hvdijk added a comment.

In D86310#3226142 <https://reviews.llvm.org/D86310#3226142>, @rnk wrote:

> In D86310#2736983 <https://reviews.llvm.org/D86310#2736983>, @hvdijk wrote:
>
>> There is a risk of bitcode incompatibilities with this change, but we 
>> already have that the code we generate now is incompatible with GCC and 
>> results in crashes that way too, I don't think there's a perfect fix, I'd 
>> like it if we could merge this. I came up with roughly the same patch today, 
>> based on current sources, to fix bug #50198 before finding this one.
>
> Who exactly generates GCC-incompatible code, clang, LLVM, or some other 
> frontend? My understanding is that Clang handles most struct layout and 
> alignment concerns in the frontend.

It's usually handled by clang, but when operations get lowered by LLVM to 
libcalls, it's coming from LLVM, and that's happening in the bug I referenced 
in that comment.


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
  • [PATCH] D86310: [X86] Alig... Reid Kleckner via Phabricator via cfe-commits
    • [PATCH] D86310: [X86]... Harald van Dijk via Phabricator via cfe-commits

Reply via email to