thakis added a comment.

In D59744#1547445 <https://reviews.llvm.org/D59744#1547445>, @wxiao3 wrote:

> @hans
>
> Please make sure all Chromium for 32-bit Linux libraries are following System 
> V ABI (i.e., m64 is passed on mmx register). I suspect that there are some 
> hand written assembly code in your libraries which is not following the ABI.


That's likely true, but also not very helpful since the ABI implications here 
are pretty big (see comments on the chromium bug). It's also currently 
impossible to write an assembly function that works with both trunk clang and 
clang 8.0.0, which makes it difficult to update compilers independent of 
changing the code. clang generally tries to be abi-compatible with itself. This 
should probably support the existing fclang-abi-compat= flag at least (and have 
a release notes entry); possibly there should be a dedicated flag for this.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59744



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

Reply via email to