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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

In D74115#1863367 <https://reviews.llvm.org/D74115#1863367>, @grimar wrote:

> 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?


Seems reasonable to me. If the section does get adopted by GNU, it likely will need to go through some standardisation (e.g. getting a new SHT_* type), so the more generic "--cg-profile" switch can be saved for then.


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

https://reviews.llvm.org/D74115