[PATCH] D98383: [Debug-Info] Add names for the debug line prologue.

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 10 21:24:10 PST 2021


ikudrin accepted this revision.
ikudrin added a comment.

LGTM, thanks!



================
Comment at: llvm/lib/MC/MCDwarf.cpp:479-482
+  MCSymbol *ProStartSym = context.createTempSymbol("prologue_start");
 
   // Create a symbol for the end of the prologue (to be set when we get there).
+  MCSymbol *ProEndSym = context.createTempSymbol("prologue_end");
----------------
These two symbols make a pair, wouldn't it be better to put them under a common comment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98383



More information about the llvm-commits mailing list