[PATCH] D16739: [CUDA] Die if we ask the NVPTX backend to emit a global ctor/dtor.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 15:56:10 PST 2016


jlebar added inline comments.

================
Comment at: lib/Target/NVPTX/NVPTXAsmPrinter.cpp:808
@@ +807,3 @@
+  const ConstantArray *InitList = dyn_cast<ConstantArray>(GV->getInitializer());
+  if (!InitList) return true;  // Not an array; we don't know how to parse.
+  return InitList->getNumOperands() == 0;
----------------
tra wrote:
> I'd make it an assert so we know if/when llvm.global_ctors mutates into something we can't deal with.
That was my inclination as well, but in AsmPrinter, this is not an error, so I'm not sure it should be one here.

https://github.com/llvm-project/llvm-project/blob/49e3b4837314fc2920616c36287aa23e4e4be6fc/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp#L1568


http://reviews.llvm.org/D16739





More information about the llvm-commits mailing list