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

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

Reply via email to