[PATCH] D11001: Add support for System z vector language extensions

Richard Smith richard at metafoo.co.uk
Wed Jul 29 18:37:01 PDT 2015

rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

LGTM, sorry for the delay. Please remove the `-mzvector` flag if we don't actually need it for GCC compatibility.

Comment at: include/clang/Driver/Options.td:1330-1333
@@ -1329,1 +1329,6 @@
+def fzvector : Flag<["-"], "fzvector">, Group<f_Group>, Flags<[CC1Option]>,
+  HelpText<"Enable System z vector language extension">;
+def fno_zvector : Flag<["-"], "fno-zvector">, Group<f_Group>,
+  Flags<[CC1Option]>;
+def mzvector : Flag<["-"], "mzvector">, Alias<fzvector>;
Does GCC really support `-mzvector`? I can't find any reference to such a flag, and grepping the GCC sources finds no occurrences of the string "zvector" at all.

Comment at: include/clang/Parse/Parser.h:115
@@ -116,1 +114,3 @@
   IdentifierInfo *Ident_bool;
+  /// Ident_pixel - cached cached IdentifierInfos for "pixel" fast comparison.
+  /// Only present if AltiVec enabled.
Too many "cached"s


More information about the cfe-commits mailing list