efriedma added a comment. In D86310#4498721 <https://reviews.llvm.org/D86310#4498721>, @hvdijk wrote:
> In D86310#4498575 <https://reviews.llvm.org/D86310#4498575>, @efriedma wrote: > >> https://reviews.llvm.org/D86310#2231136 has an example where IR generated by >> clang breaks. > > clang bases it on the data layout, so when the change here is applied, clang > already generates correct IR for that example without further changes (using > `%struct.Y = type <{ i64, %struct.X }>`). Unless you were using that as an > example of when using old clang to generate LLVM IR, and new LLVM to produce > machine code, would break? I only meant to dispute the assertion that ABI compatibility with old IR is only a problem for non-clang frontends. 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
