[PATCH] D28825: [NVPTX] Support global variables of integer type larger than i64.

[Justin Lebar via Phabricator via llvm-commits]

jlebar marked an inline comment as done.
jlebar added inline comments.


================
Comment at: llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp:2002
+    APInt Val = CI->getValue();
+    for (unsigned I = 0, E = DL.getTypeStoreSize(CPV->getType()); I < E; ++I) {
+      uint8_t Byte = Val.getLoBits(8).getZExtValue();
----------------
majnemer wrote:
> Do we want the store size or the allocation size?
Ah, indeed.


https://reviews.llvm.org/D28825