[PATCH] D12414: [NVPTX] add an NVPTX-specific alias analysis

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 6 21:47:37 PDT 2015


hfinkel added inline comments.

================
Comment at: lib/Target/NVPTX/NVPTXAliasAnalysis.cpp:71
@@ +70,3 @@
+      if (auto A = dyn_cast<Argument>(Obj)) {
+        if (TM && TM->getDrvInterface() == NVPTX::CUDA &&
+            isKernelFunction(*A->getParent()))
----------------
Can you please explain why this DvrInterface == CUDA check is necessary? isKernelFunction always checks for NVVM metadata, and lacking that, a specific calling convention. Is that not enough?



http://reviews.llvm.org/D12414





More information about the llvm-commits mailing list