[PATCH] D108019: [NFC][llvm-xray] add a llvm-xray convert option `symbolize-no-demangle`
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 15 19:56:12 PDT 2021
MaskRay added inline comments.
================
Comment at: llvm/test/tools/llvm-xray/X86/convert-with-debug-syms-no-demangle.txt:1
+; RUN: llvm-xray convert -m %S/Inputs/elf64-sample-o2.bin --symbolize --symbolize-no-demangle=true %S/Inputs/naive-log-simple.xray -f=yaml -o - 2>&1 | FileCheck %s
+
----------------
Enna1 wrote:
> MaskRay wrote:
> > This can be added to `X86/extract-instrmap-symbolize.ll` rather than add a new file.
> > This can be added to `X86/extract-instrmap-symbolize.ll` rather than add a new file.
>
> `llvm xray convert` and `llvm xray extract` are different subcommand, to avoid adding a new file, perhaps add this to `X86/convert-with-debug-syms.txt` ?
SG.
Please also avoid options like `--foo=true`.
It works just because llvm CommandLine.cpp has such behaviors. Such options are not guaranteed to work in the future for user-facing utilities.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108019/new/
https://reviews.llvm.org/D108019
More information about the llvm-commits
mailing list