[PATCH] D70759: [llvm-symbolizer] Support debug file lookup using build ID
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 4 11:28:07 PST 2019
MaskRay added inline comments.
================
Comment at: llvm/lib/DebugInfo/Symbolize/Symbolize.cpp:328
+ sys::path::append(Path, ".build-id",
+ llvm::toHex(BuildID[0], /*LowerCase*/ true),
+ llvm::toHex(BuildID.slice(1), /*LowerCase*/ true));
----------------
Super nit: `/*LowerCase=*/` may be more common. At least clang-format recognizes this form and deletes the space before `true`.
================
Comment at: llvm/test/DebugInfo/symbolize-build-id.test:1
+# RUN: yaml2obj %s > %t
+
----------------
In binary utilities tests, we use `-o` to avoid shell redirection.
================
Comment at: llvm/test/DebugInfo/symbolize-build-id.test:15
+FileHeader:
+ Class: ELFCLASS64
+ Data: ELFDATA2LSB
----------------
`Machine:` is the longest key. The indentation of values can be reduced to the minimum.
`Machine: EM_X86_64`
================
Comment at: llvm/test/DebugInfo/symbolize-build-id.test:23
+ Flags: [ SHF_ALLOC ]
+ Content: 040000000800000003000000474e5500abb50d82b6bdc861
+ProgramHeaders:
----------------
ditto. `Content: 040000000800000003000000474e5500abb50d82b6bdc861`
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70759/new/
https://reviews.llvm.org/D70759
More information about the llvm-commits
mailing list