[PATCH] D74611: [obj2yaml][WIP] - Fix indentations in the ELF output.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 18 04:02:41 PST 2020


jhenderson added a comment.

In D74611#1880357 <https://reviews.llvm.org/D74611#1880357>, @grimar wrote:

> In D74611#1880193 <https://reviews.llvm.org/D74611#1880193>, @jhenderson wrote:
>
> > Also, should this really be restricted to ELF?
>
>
> I see no obvious reasons for such restrictions. Though I supposed that we might want to start from ELF and then
>  enable it for other targets. Do you have a preference here? I.e. do you want to see it for all targets from the begining?


I don't have a strong preference, and am happy for you to go with whatever is easier.


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

https://reviews.llvm.org/D74611





More information about the llvm-commits mailing list