[PATCH] D49754: Add -m(no-)spe, and e500 CPU definitions and support to clang

Tue Feb 19 12:02:31 PST 2019

vit9696 added a comment.

This is a series of patches, which I believe should merged altogether. Currently the following patches are relevant:

- https://reviews.llvm.org/D49754
- https://reviews.llvm.org/D54409
- https://reviews.llvm.org/D54583
- https://reviews.llvm.org/D56703

The patches are intended to add PowerPC SPE support, and they do not seem to break things outside. Initially I wanted them to get merged into 8.x, but extensive local testing unveiled a number of issues, so the time was missed. I believe all the 4 patches are pretty much ready for merging, aside the following:

- approval from @jhibbits and @kthomsen, who do some testing as well
- https://reviews.llvm.org/D49754#1401168 is fixed or declared a separate issue, handled outside of the patchset

Depending on the above the decision should be taken. Meanwhile, a test should be added for `__NO_FPRS__` near `PPC32-SPE:#define __SPE__ 1 ` now that we define it.

Repository:
  rC Clang

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

https://reviews.llvm.org/D49754