[PATCH] [PowerPC] Initial VSX intrinsic support, with min/max for vector double

Bill Schmidt wschmidt at linux.vnet.ibm.com
Thu Oct 30 10:23:04 PDT 2014


Thanks for the suggestion, Hal -- this looks much cleaner to me.  As a side benefit, I was able to clear up the weird behavior of giving both an error and a not-really-pertinent warning for "vector long double."  There also wasn't an issue with the caret position, as I was able to just use the TST location in DeclSpec::Finish.

Let me know whether the wording of the new error message for vector double is ok.

http://reviews.llvm.org/D5949

Files:
  include/clang/Basic/BuiltinsPPC.def
  include/clang/Basic/DiagnosticParseKinds.td
  lib/Basic/Targets.cpp
  lib/Headers/altivec.h
  lib/Sema/DeclSpec.cpp
  test/CodeGen/builtins-ppc-vsx.c
  test/Parser/altivec.c
  test/Parser/cxx-altivec.cpp
  test/Parser/vsx.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5949.15573.patch
Type: text/x-patch
Size: 11812 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141030/47aa972d/attachment.bin>


More information about the cfe-commits mailing list