[PATCH] D70508: Fix an offset underflow bug in DwarfExpression when describing small values with subregisters

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 13:39:29 PST 2019


vsk added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfExpression.cpp:168
+    // Mark it as emitted.
+    Coverage.set(Offset, Offset + Size);
+    CurPos = Offset + Size;
----------------
Is it a requirement that all bits in `MachineReg` be described? What happens is there's an unset bit in `Coverage`?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70508/new/

https://reviews.llvm.org/D70508





More information about the llvm-commits mailing list