tra added a comment.

In D118977#3299974 <https://reviews.llvm.org/D118977#3299974>, @jchlanda wrote:

>> Target ISA Notes
>> Requires sm_53 or higher.

I think we do need this constraint applied to the new builtins, too. Right now 
nothing stops using them on a GPU where they do not exist and that will likely 
crash the compiler when we fail to find a matching intrinsic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118977

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

Reply via email to