arsenm added a comment.
I think this may not hard break mesa. I believe mesa bypasses the intrinsic
creation API, and just declares the string name of the intrinsic. The type name
mangling suffix is technically irrelevant, and as long as you use a consistent
type with a consistent suffix things should work out (and the null suffix also
works). After committing mesa should still move to adding the type suffix
================
Comment at: llvm/test/CodeGen/AMDGPU/llvm.amdgcn.readlane.ll:13
define amdgpu_kernel void @test_readlane_sreg_sreg(i32 %src0, i32 %src1) #1 {
%readlane = call i32 @llvm.amdgcn.readlane(i32 %src0, i32 %src1)
call void asm sideeffect "; use $0", "s"(i32 %readlane)
----------------
This is a hint mesa won't break
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147732/new/
https://reviews.llvm.org/D147732
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
