[PATCH] D101761: [llvm-objdump] support "-M att" and "-M intel" flags

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 3 10:51:09 PDT 2021


MaskRay added a comment.

In D101761#2733856 <https://reviews.llvm.org/D101761#2733856>, @thakis wrote:

> In D101761#2733553 <https://reviews.llvm.org/D101761#2733553>, @MaskRay wrote:
>
>> D101695 <https://reviews.llvm.org/D101695>
>
> Oh, cool. Your patch doesn't get interleaving both kinds of options right though afaict.

I don't think that is necessary. Users shouldn't use `--x86-asm-syntax`. Actually we probably should remove it afterwards.


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

https://reviews.llvm.org/D101761



More information about the llvm-commits mailing list