[PATCH] D51640: [DebugInfo] Normalize common kinds of DWARF sub-expressions.

[Paul Robinson via Phabricator via llvm-commits]

probinson added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfExpression.cpp:31
+    emitOp(dwarf::DW_OP_lit0 + Value);
+  else if (Value == ~0U) {
+    emitOp(dwarf::DW_OP_lit0);
----------------
aprantl wrote:
> These are 64 bit values. We either need ~0ULL or std::limits::max<uint64_t>().
The DWARF expression stack uses target-address-size values, and that isn't necessarily 64-bit.  I'm not sure we can make this compaction work in all cases.


https://reviews.llvm.org/D51640