[PATCH] D18700: [Inline asm][GCC compatibility] Handle %v-prefixed code in inline assembly

[Eric Christopher via cfe-commits]

echristo added a comment.

That's fascinating and terrifying all at once.

Can you rename the test to avx-v-modifier-inline-asm.c please?

One inline comment as well.

Thanks!

-eric


================
Comment at: lib/AST/Stmt.cpp:639
@@ +638,3 @@
+      // into "pcmpestri" otherwise.
+      if (C.getTargetInfo().hasFeature("avx"))
+        CurStringPiece = "v" + CurStringPiece;
----------------
This should probably check the feature set based on the function rather than the feature set of the base compile yes? What does gcc do if you put it in an __attribute__((target("..."))) function?


http://reviews.llvm.org/D18700