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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to