[PATCH] D101847: [AMDGPU] Fix function pointer argument bug in AMDGPU Propagate Attributes pass.
Madhur Amilkanthwar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 13 11:20:15 PDT 2021
madhur13490 added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUPropagateAttributes.cpp:252
CallBase *CI = dyn_cast<CallBase>(I);
- if (!CI)
+ if (!CI || CI->getCalledOperand() != &F)
continue;
----------------
A comment here, please!
================
Comment at: llvm/test/CodeGen/AMDGPU/propagate-attributes-function-pointer-argument.ll:14
+; CHECK: define amdgpu_kernel void @thiswasabug() #0
+; CHECK: call void @f(void ()* @g.1)
+; CHECK: call void @g()
----------------
madhur13490 wrote:
> Probably can have CHECK-NOT too to explicitly call out what should not be generated.
CHECK-LABEL followed by CHECK-DAG would be more appropriate here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101847/new/
https://reviews.llvm.org/D101847
More information about the llvm-commits
mailing list