[PATCH] D109178: [PowerPC] Disable vector types when not supported by subtarget features

Nemanja Ivanovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 7 07:25:31 PDT 2021


nemanjai added a comment.

It is not clear what this patch will do with `vector [[un]signed] long`. This is currently deprecated but still works - and we don't want it to work with no VSX.



================
Comment at: clang/lib/Headers/altivec.h:8881
 }
 #endif
 
----------------
This file is huge and there is a lot of conditional compilation within it. Can you please add comments to terminating `endif` as to what it is terminating. Presumably this one here terminates `__VSX__`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109178



More information about the cfe-commits mailing list