craig.topper added a comment.

In D87528#2357043 <https://reviews.llvm.org/D87528#2357043>, @mibintc wrote:

> Actually kludging it by just removing the assert isn't going to work. I'll 
> ping Pengfei to see about developing a patch for this problem.

This is likely a WebAssembly backend problem. Each backend needs to do work to 
support constrained intrinsics. D80952 <https://reviews.llvm.org/D80952> added 
a way to know if a target supports constrained intrinsics. Probably need to 
apply that to the changes here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87528

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

Reply via email to