[PATCH] D26982: Restructure DwarfDebug::beginInstruction (NFC).

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 22 10:53:26 PST 2016


dblaikie accepted this revision.
dblaikie added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfDebug.cpp:1016-1037
+  if (DL) {
+    // We have a new, explicit location.
+    unsigned Flags = 0;
+    PrevInstLoc = DL;
+    if (DL == PrologEndLoc) {
+      Flags |= DWARF2_FLAG_PROLOGUE_END;
+      PrologEndLoc = DebugLoc();
----------------
Would it help to reverse these two and early return from the UnknownLocations case - so the major "if (DL)" case could be non-indented? (but if the way you've got it works best for your other patch, that's cool)


https://reviews.llvm.org/D26982





More information about the llvm-commits mailing list