[PATCH] D96409: [debug-info] refactor emitDwarfUnitLength

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/MC/MCStreamer.cpp:1013-1014
   AddComment(Comment);
+  MCSymbol *Lo = Context.createTempSymbol(Prefix + "start");
+  MCSymbol *Hi = Context.createTempSymbol(Prefix + "end");
+
----------------
ikudrin wrote:
> Maybe use `"_start"` and `"_end"` here and avoid passing the trailing `'_'` in the argument?
hmm, if `Prefix` is passed as `""`, will it be a little strange to have a label like: `.L_start0`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96409