https://github.com/epilk closed https://github.com/llvm/llvm-project/pull/79905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -25,4 +25,4 @@ entry:
}
!llvm.module.flags = !{!0}
-!0 = !{i32 1, !"amdgpu_code_object_version", i32 500}
+!0 = !{i32 1, !"amdhsa_code_object_version", i32 500}
epilk wrote:
Sure, I'll make a PR for that too after this lands.
https://github.com/llvm/llvm-
epilk wrote:
@tstellar there was some uncertainty about whether to leave the spelling as-is
since mesa used to following HSA code object versions (but doesn't anymore).
Would you have any objections to this change?
https://github.com/llvm/llvm-project/pull/79905
___
@@ -177,16 +177,19 @@ void justAtAvailable(void) {
#ifdef OBJCPP
-int f(char) AVAILABLE_10_12;
+int f(char) AVAILABLE_10_12; // #f_char_def
int f(int);
template int use_f() {
- // FIXME: We should warn here!
- return f(T());
epilk wrote:
Could you che
epilk wrote:
Hello, apologies for jumping in here so late, but this commit is changing the
device function ABI on AMDGPU. For instance, for a device function returning
this struct:
```
struct Empty {};
struct RetTy {
int f1[3];
Empty e;
int f2;
};
__device__ RetTy deviceFn() { ... }
```
