[PATCH] D52741: [AMDGPU] Add an AMDGPU pass to promote bitcast calls

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 3 02:14:22 PDT 2018


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUFixFunctionBitcasts.cpp:35
+public:
+  void visitCallInst(CallInst &CI) {
+    CallSite CS(&CI);
----------------
Should use visitCallSite to handle invokes as well


================
Comment at: lib/Target/AMDGPU/AMDGPUFixFunctionBitcasts.cpp:37-38
+    CallSite CS(&CI);
+    if (!CS.getCalledValue())
+      return;
+    if (CS.getCalledFunction())
----------------
How can this happen?


https://reviews.llvm.org/D52741





More information about the llvm-commits mailing list