[PATCH] D118273: [BOLT][DWARF] Fix gdb index section

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 26 13:10:12 PST 2022


ayermolo added inline comments.


================
Comment at: bolt/test/X86/gdbindex.test:1
+RUN: rm -rf %t
+RUN: mkdir %t
----------------
yota9 wrote:
> Just curious is this necessary? Also is the %T (directory) meant? 
> Also should we convert binaries to yaml?
Yes %t refers to a directory. It's nice to start with in clean state I think.
I think it's fine. This is basically re-purposed test from llvm/test/DebugInfo


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118273



More information about the llvm-commits mailing list