[PATCH] D143467: [PowerPC] Add target feature requirement to builtins

[Qiu Chaofan via Phabricator via cfe-commits]

qiucf added a comment.

In D143467#4182457 <https://reviews.llvm.org/D143467#4182457>, @nemanjai wrote:

> Another note regarding the motivating test case:
> Use of `vector double` should require VSX. We don't really seem to have the ability to turn this off early enough to catch this though. It would seem that in the front end, the target features depend on the compilation options and not the function itself.

Thanks for reminding me of `vector double` exception! I have D143479 <https://reviews.llvm.org/D143479> to make target feature check respect target attributes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143467