[PATCH] D113707: [clang] Make -masm=intel affect inline asm style

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 11 13:19:59 PST 2021


rnk added a comment.

This seems reasonable, but I worry about headers that contain AT&T style inline asm blobs, such as clang's own intrinsic headers. As you say, the directive is local. I would hate to end up in a place where we have to prefix every asm blob in clang/lib/Headers/*mmintrin.h with ".att_syntax on;".

Actually, `git grep __asm__ ../clang/lib/Headers` shows that there are not very many blobs. Maybe we should just prefix them.


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

https://reviews.llvm.org/D113707



More information about the cfe-commits mailing list