https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From f0d1f76540ca2121a7b70c71c7403a7d45584482 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/8] Pre-commit test: [Pr
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From 813e65b376216543f0250270f2b59ef888adf35d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/6] [Clang][OpenCL] Pre-
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/144850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez wrote:
> @jmmartinez It appears that CUDA tests are broken by this change, because it
> uses `noexcept` which is a C++11 feature not available with c++98:
> https://lab.llvm.org/buildbot/#/builders/69/builds/22562/steps/8/logs/stdio
>
> ```
> FAILED:
> External/CUDA/CMakeFiles/algo
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/144850
Reverts llvm/llvm-project#136133
From 366be49837a1fa669d836cc167b1b6dbf2725110 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 19 Jun 2025 09:11:36 +0200
Subjec
jmmartinez wrote:
We have now a test for std::array on llvm-test-suite. Is it ok if we land this
now ?
I have yet to update the buildbot to run with the latest gcc-toolchain. I've
played with using spack to install it but it doesn't have gcc-15 toolchain
available yet.
https://github.com/llv
@@ -84,6 +84,8 @@ class FrontendAction {
/// \return True on success; on failure ExecutionAction() and
/// EndSourceFileAction() will not be called.
virtual bool BeginSourceFileAction(CompilerInstance &CI) {
+if (CurrentInput.isPreprocessed())
+ CI.getPreprocesso
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From f0d1f76540ca2121a7b70c71c7403a7d45584482 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/8] Pre-commit test: [Pr
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=C
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From fa4c334c13e682e9f65d5f9b523deca588f8da6d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/6] [Clang][OpenCL] Pre-
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136133
From 29463adcd2869c9a6777d54d3521f0d435e3d38c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 17 Apr 2025 13:41:55 +0200
Subject: [PATCH] [CUDA][HIP] Add a __devi
@@ -84,6 +84,8 @@ class FrontendAction {
/// \return True on success; on failure ExecutionAction() and
/// EndSourceFileAction() will not be called.
virtual bool BeginSourceFileAction(CompilerInstance &CI) {
+if (CurrentInput.isPreprocessed())
+ CI.getPreprocesso
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=C
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From 82968e8ae497bf40533487c6a8c11a50e4a1e279 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/5] [Clang][OpenCL] Pre-
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=C
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=C
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From e9940ff50bed67583d3f285a4adf105e7ab08547 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/4] [Clang][OpenCL] Pre-
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From e9940ff50bed67583d3f285a4adf105e7ab08547 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/4] [Clang][OpenCL] Pre-
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From e9940ff50bed67583d3f285a4adf105e7ab08547 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/4] [Clang][OpenCL] Pre-
@@ -0,0 +1,99 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// expected-no-diagnostics
jmmartinez wrote:
Thanks! I thought it only applied to the last RUN lin
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/143507
From e9940ff50bed67583d3f285a4adf105e7ab08547 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/4] [Clang][OpenCL] Pre-
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/143507
None
From f5deb5fd45d9ac5b03e610e1234e98a57c93f8e5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 10 Jun 2025 12:28:35 +0200
Subject: [PATCH 1/3] [Clang][OpenCL
jmmartinez wrote:
Hello ! I’m still exploring possible alternatives to this PR but I haven't
found anything satisfying yet. If you have some suggestions don't hesitate.
Thanks !
https://github.com/llvm/llvm-project/pull/137665
___
cfe-commits maili
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From c9bff12d85d4d589cf31459ddabba2387ca83852 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/6] Pre-commit test: [Pr
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From f416b07520e73c7feec72378d970305e85295ad8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/6] Pre-commit test: [Pr
@@ -1558,6 +1558,7 @@ void clang::InitializePreprocessor(Preprocessor &PP,
const PCHContainerReader &PCHContainerRdr,
const FrontendOptions &FEOpts,
const CodeGenOptions &Co
@@ -84,6 +84,8 @@ class FrontendAction {
/// \return True on success; on failure ExecutionAction() and
/// EndSourceFileAction() will not be called.
virtual bool BeginSourceFileAction(CompilerInstance &CI) {
+if (CurrentInput.isPreprocessed())
+ CI.getPreprocesso
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From 002795597e3c1c887e5099f0efa7e6b23eefad8e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/6] Pre-commit test: [Pr
@@ -1558,6 +1558,13 @@ void clang::InitializePreprocessor(Preprocessor &PP,
const PCHContainerReader &PCHContainerRdr,
const FrontendOptions &FEOpts,
const CodeGenOptions &C
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
From 72bc06251047bb92a5bc6a068670b047c078a3c3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Mon, 28 Apr 2025 17:05:46 +0200
Subject: [PATCH 1/4] Pre-commit test: [Pr
jmmartinez wrote:
Ping !
https://github.com/llvm/llvm-project/pull/137665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez approved this pull request.
Thanks for the fix !
https://github.com/llvm/llvm-project/pull/140490
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/139697
From e60d2b04d1cb7873e4034cb639406b1c329d1c45 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Tue, 13 May 2025 10:32:41 +0200
Subject: [PATCH] [HIP] Fix return type in
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/139891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez wrote:
> This seems reasonable. Can you update the description to explain the
> motivation for the change, though? Is it a bug you are trying to fix, or a
> specific scenario you think should be optimized? Thanks
I've updated the description with an explanation of the situation that
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/137665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/139892
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/139697
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/139892
ambibuity -> ambiguity
From 5509f15dde030de212660a394d9b98530eda3681 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Wed, 14 May 2025 13:40:42 +0200
Subject: [PATCH]
jmmartinez wrote:
Added a tests and created a separate PR to commit the tests before the fix in
https://github.com/llvm/llvm-project/pull/139891
https://github.com/llvm/llvm-project/pull/139697
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
h
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/139891
Tests related to https://github.com/llvm/llvm-project/pull/139697
From 57abd4e1a54aedaeeaaceca394f298d7f7abc4eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Wed, 1
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/137665
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
jmmartinez wrote:
I fixed the `Modules/initializers.cpp` test by moving the preprocessor
directives outside of the module contents (the module contents should be
already preprocessed and not contain preproc directives).
https://github.com/llvm/llvm-project/pull/137665
_
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/139697
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/139697
Before, some functions like `isgreater(double, double)` would return a `double`
instead of a `bool`.
I've stumbled upon this bug while trying to adapt
[`External/CUDA/cmath.cu`](https://github.com/llvm/llvm
jmmartinez wrote:
> Now the question is -- how do we test it?
I'm working on adding some tests for the cuda_wrapper headers:
* [ ] First in https://github.com/llvm/llvm-test-suite/tree/main/External/CUDA
(there are some already related to other cuda_wrapper headers)
* [ ] and likely something v
jmmartinez wrote:
> module maps don't get preprocessed, so I don't think it matters what that is
> set to. For normal module.modulemap files `Preprocessed` is `false`. I think
> it's fine to remove the true and see if anything breaks, the preprocessor
> just skips over that section anyway.
Cu
https://github.com/jmmartinez approved this pull request.
Thanks @spavloff ! I'm sorry I didn't address the remarks earlier, I was
out-of-office and didn't see the notifications.
https://github.com/llvm/llvm-project/pull/139492
___
cfe-commits mailing
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/137623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/136853
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez wrote:
Thanks !
https://github.com/llvm/llvm-project/pull/137306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/137306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez wrote:
I've added a verbose version of the assertion that uses `__builtin_printf` (I
could not figure out a way to correctly emit something to `stderr` from device
code in a generic way without including `cstdio`.
```cpp
#ifdef _GLIBCXX_VERBOSE_ASSERT
__attribute__((device, noreturn
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136133
From a602d4f56993c2fc2825997a367b4551fcd54817 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 17 Apr 2025 13:41:55 +0200
Subject: [PATCH] [CUDA][HIP] Add a __devi
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136133
From 2136a05f0b8a0f2db0f9506a78ebf8f41c0f4d2f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 17 Apr 2025 13:41:55 +0200
Subject: [PATCH] [CUDA][HIP] Add a __devi
@@ -155,3 +155,11 @@
#endif
#pragma OPENCL EXTENSION cl_amd_media_ops2: enable
+#if (__OPENCL_C_VERSION__ >= 300)
jmmartinez wrote:
Thanks ! You're right. It's available by default on OpenCL C 2 but not 3.
https://github.com/llvm/llvm-project/pull/137636
___
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/137665
This is a draft while I'm trying to figure out what's left to do.
This has issues with the test `Modules/initializers.cpp`.
The function `CompilerInstance::createModuleFromSource` creates a
`FrontendInput`
@@ -155,3 +155,11 @@
#endif
#pragma OPENCL EXTENSION cl_amd_media_ops2: enable
+#if (__OPENCL_C_VERSION__ >= 300)
jmmartinez wrote:
I don't understand, isn't `__opencl_c_generic_address_space` available by
default with `OPENCL_C_VERSION >= 200` (from reading
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/137623
`-rewrite-objc` passes `-x objective-c++-cpp-output` as input type to the
preprocessor job. This is not correct since we would be preprocessing a
preprocessed file. The correct input type is `objective-c++`.
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/137306
When using `-dD` to generate a preprocessed output, the `#define` directives
are preserved. This includes built-in and command-line definitions.
Before, clang would warn for reserved identifiers for serialize
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136853
From 1acf00c794bddd5aed30670915a83db5b243b7ff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Wed, 16 Apr 2025 14:20:53 +0200
Subject: [PATCH 1/2] Pre-commit test: [Cl
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136133
From b433add1b7c2027b89cdd9be11680c0a9a59e807 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 17 Apr 2025 13:41:55 +0200
Subject: [PATCH] [CUDA][HIP] Add a __devi
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/136853
When using -no-integrated-cpp, before, the driver won't collapse actions when
the input was not llvm-ir
or it would collapse them too aggressively with -save-temps
The original code was checking the action t
@@ -0,0 +1,35 @@
+// libstdc++ uses the non-constexpr function std::__glibcxx_assert_fail()
+// to trigger compilation errors when the __glibcxx_assert(cond) macro
+// is used in a constexpr context.
+// Compilation fails when using code from the libstdc++ (such as std::array) on
@@ -0,0 +1,35 @@
+// libstdc++ uses the non-constexpr function std::__glibcxx_assert_fail()
+// to trigger compilation errors when the __glibcxx_assert(cond) macro
+// is used in a constexpr context.
+// Compilation fails when using code from the libstdc++ (such as std::array) on
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/136133
From b98abc3e1b1ca6c81509e310b6df1f7f7cfd10b9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 17 Apr 2025 13:41:55 +0200
Subject: [PATCH] [CUDA][HIP] Add a __devi
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/136133
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,35 @@
+// libstdc++ uses the non-constexpr function std::__glibcxx_assert_fail()
+// to trigger compilation errors when the __glibcxx_assert(cond) macro
+// is used in a constexpr context.
+// Compilation fails when using code from the libstdc++ (such as std::array) on
jmmartinez wrote:
See
https://github.com/gcc-mirror/gcc/blob/c237297ee596545552f48ec9582d7d16703d8949/libstdc%2B%2B-v3/include/bits/c%2B%2Bconfig#L645-L659
for the problematic libstdc++ snippet
https://github.com/llvm/llvm-project/pull/136133
___
cfe
jmmartinez wrote:
This patch is cruelly missing some tests. Is there a place for tests of this
kind? I haven't found an obvious one for other headers.
https://github.com/llvm/llvm-project/pull/136133
___
cfe-commits mailing list
cfe-commits@lists.llvm
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/136133
libstdc++ 15 uses the non-constexpr function
std::__glibcxx_assert_fail() to trigger compilation errors when the
__glibcxx_assert(cond) macro is used in a constantly evaluated context.
Compilation fails when
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/135027
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/135376
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/135376
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/135376
The vmem-to-lds-loads-insts feature is only available on gfx9/10. While
target-parser was also enabling it for gfx6,7,8.
From 90c95af57317103f399ee4059a4e4ec01ddc98de Mon Sep 17 00:00:00 2001
From: =?UTF-8?q
jmmartinez wrote:
> IIRC there were bugs when you try to use lambda in conjunction with the
> target attribute. Can you add a test where you try to use a builtin that
> depends on a subtarget feature, in a function marked with the target
> attribute (and not available in the main compile targe
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/135027
From a22e56d063e91df667d89ed5238bd886ce10bd0a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Wed, 9 Apr 2025 15:11:19 +0200
Subject: [PATCH] [Clang][AMDGPU] Accept bu
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/135027
From 4e5736d1b90ae477642ea03dcf39231f5f0a8e59 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Wed, 9 Apr 2025 15:11:19 +0200
Subject: [PATCH 1/4] [Clang][AMDGPU] Accep
@@ -0,0 +1,24 @@
+// REQUIRES: amdgpu-registered-target
+// RUN: %clang_cc1 -std=c++20 -triple amdgcn -target-cpu tahiti -fsyntax-only
-fcuda-is-device -verify=expected -o - %s
+// RUN: %clang_cc1 -std=c++20 -triple amdgcn -target-cpu gfx950 -fsyntax-only
-fcuda-is-device -o - %
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/135027
`Sema::getCurFunctionDecl(AllowLambda = false)` returns a nullptr when the
lambda declaration is outside a function (for example, when assigning a lambda
to a static constexpr variable).
This triggered an a
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/134598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez wrote:
Thanks for the quick review !
https://github.com/llvm/llvm-project/pull/134598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/134598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/134598
The pointer argument for `wait_for_event(int, event_t*)` should take the
default address space: generic if available, otherwise private.
Before this patch it would always be generic with `-fdeclare-opencl-bu
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/132048
From 27b90ab19cc8c8694784b41b53b2623c718071ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/3] [AMDGPU] Add "lds-bu
jmmartinez wrote:
I've added a commit to have a common attribute that we could use for both,
global-load-lds and buffer-load-lds builtins.
https://github.com/llvm/llvm-project/pull/133055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133741
From 41af38793161b0f1535c98c4695c36e081ef2f67 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 27 Mar 2025 17:46:34 +0100
Subject: [PATCH 01/10] [Clang][AMDGPU] Ad
https://github.com/jmmartinez closed
https://github.com/llvm/llvm-project/pull/132048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/132048
From 5f039ef32105132a7edf6203eb0b7825d4b8b503 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/6] [AMDGPU] Add "lds-bu
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133055
From 5f039ef32105132a7edf6203eb0b7825d4b8b503 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/5] [AMDGPU] Add "lds-bu
@@ -729,6 +729,25 @@ GCNTTIImpl::instCombineIntrinsic(InstCombiner &IC,
IntrinsicInst &II) const {
break;
}
+ case Intrinsic::amdgcn_cvt_off_f32_i4: {
+Value* Arg = II.getArgOperand(0);
+Type *Ty = II.getType();
+
+if (isa(Arg))
+ return IC.replaceInst
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133741
From 41af38793161b0f1535c98c4695c36e081ef2f67 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 27 Mar 2025 17:46:34 +0100
Subject: [PATCH 1/9] [Clang][AMDGPU] Add
https://github.com/jmmartinez created
https://github.com/llvm/llvm-project/pull/133741
This built-in that maps to `V_CVT_OFF_F32_I4` which treats its input as a 4-bit
signed integer and returns `0.0625f * src`.
SWDEV-518861
From 41af38793161b0f1535c98c4695c36e081ef2f67 Mon Sep 17 00:00:00 200
@@ -260,7 +260,7 @@ AMDGPUTargetInfo::AMDGPUTargetInfo(const llvm::Triple
&Triple,
MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 64;
CUMode = !(GPUFeatures & llvm::AMDGPU::FEATURE_WGP);
- for (auto F : {"image-insts", "gws"})
+ for (auto F : {"image-insts", "gws", "mem
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133055
From 93232bb8255e927d62dd9a5f63e885e73ccf0eb5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/5] [AMDGPU] Add "lds-bu
@@ -0,0 +1,45 @@
+; RUN: split-file %s %t
+;
+; RUN: not --crash llc -mtriple=amdgcn -mcpu=gfx1100 %t/struct.ll 2>&1 |
FileCheck --ignore-case %s
+; RUN: not --crash llc -global-isel -mtriple=amdgcn -mcpu=gfx1100 %t/struct.ll
2>&1 | FileCheck --ignore-case %s
+; RUN: not --crash
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133055
From bf3f06cc99a1ac2460ffea741c0fe495f0778e70 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/4] [AMDGPU] Add "lds-bu
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133055
From bf3f06cc99a1ac2460ffea741c0fe495f0778e70 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/3] [AMDGPU] Add "lds-bu
https://github.com/jmmartinez updated
https://github.com/llvm/llvm-project/pull/133055
From bf3f06cc99a1ac2460ffea741c0fe495f0778e70 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Juan=20Manuel=20Martinez=20Caama=C3=B1o?=
Date: Thu, 20 Mar 2025 13:23:25 +0100
Subject: [PATCH 1/3] [AMDGPU] Add "lds-bu
https://github.com/jmmartinez edited
https://github.com/llvm/llvm-project/pull/133055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 177 matches
Mail list logo
