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

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

Reply via email to