[PATCH] D139750: Optionally print symbolizer markup backtraces.

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: llvm/lib/Support/Unix/Signals.inc:538
+      OS << format("{{{mmap:%#016x:%#x:load:%d:%s:%#016x}}}\n", StartAddress,
+                   Phdr->p_memsz, ModuleCount, (char *)ModeStr,
+                   ModuleRelativeAddress);
----------------
Can you use `reinterpret_cast` instead of C-style cast?


================
Comment at: llvm/lib/Support/Unix/Signals.inc:561
+      ArrayRef<uint8_t> Notes(
+          (const uint8_t *)(Info->dlpi_addr + Phdr->p_vaddr), Phdr->p_memsz);
+      while (Notes.size() > 12) {
----------------
Can you use C++-style instead of C-style cast here and below?


================
Comment at: llvm/lib/Support/Unix/Signals.inc:571
+        ArrayRef<uint8_t> Name = Notes.take_front(NameSize);
+        uint32_t AmtToSkip =
+            alignToPowerOf2((uintptr_t)Notes.data() + NameSize, 4) -
----------------
What does `Amt` refer to? Might be better to use something more self-descriptive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139750