FreddyYe added a comment.

In D139170#3965814 <https://reviews.llvm.org/D139170#3965814>, @mgehre-amd 
wrote:

> Do other targets not support > 128 bit integers, or is this PR only the first 
> conservative step of lifting the limit?

I temporary only enabled > 128 bit FP conversion for X86 in 
https://reviews.llvm.org/D137241, since I highly relied on end-to-end tests to 
implement that pass and I don't have other targets' environment to verify. So 
same to this patch, I only lift the limit for X86. Hope other targets will 
follow us to enable for themselves. WDYT?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139170

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

Reply via email to