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

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 3 12:51:20 PDT 2021


thakis abandoned this revision.
thakis added a comment.

Conceptually, it seems a bit nicer to me to not add the options to DisassemblerOptions in the first place instead of adding and then removing them again. But I mostly want the feature and don't care too much :) So abandoning this in favor of your CL.

Consider stealing the code bits in this CL for your CL?


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

https://reviews.llvm.org/D101761



More information about the llvm-commits mailing list