[PATCH] D74115: [llvm-readobj][test] - Add a test for --elf-cg-profile option.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 7 01:29:41 PST 2020


grimar added a comment.

In D74115#1863351 <https://reviews.llvm.org/D74115#1863351>, @jhenderson wrote:

> Aside: why is this switch prefixed with "elf"? I wouldn't think this section is ELF-specific in principle...


Hmmm. Should we rename to `--llvm-cg-profile`? Its a LLVM extention first of all.
And if we want to implement the GNU version (that is what I am working on), perhaps it is good idea to all this suffix to emphasise that is not a common GNU option?


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

https://reviews.llvm.org/D74115





More information about the llvm-commits mailing list