[PATCH] D126835: Round up zero-sized symbols to 1 byte in `.debug_aranges` (without breaking other logic).

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 10:42:07 PDT 2022


ayermolo added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:3056
         // write a single arange entry containing just that one symbol.
-        uint64_t Size = SymSize[Span.Start];
+        auto Size = SizeRef->second;
         if (Size == 0)
----------------
Can you change back to unit64_t. I believe LLVM coding standard errs on using specific types when it's not obvious what type it is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126835



More information about the llvm-commits mailing list