nikic added a comment.

I'm happy to sign off on the x86-64 part here, but I'm less sure about x86-32. 
If I understood correctly, the i128 alignment is raised there exclusively to 
fix the "f128 legalized to i128 libcall" case -- is there any other ABI 
requirement for i128 alignment on x86-32? Is raising i128 alignment the right 
way to fix an f128 issue?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86310

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

Reply via email to