Hahnfeld added a comment. In https://reviews.llvm.org/D42168#978593, @ABataev wrote:
> After some investigation, I think this whole code is incorrect. We should not > rely on the fact that there is no padding on some architectures and should > mix the type generated by the Clang and initial value, generated by LLVM. I'm not sure I understand what you mean. Do you refer to `auto Align = CharUnits::fromQuantity(1)`? Repository: rC Clang https://reviews.llvm.org/D42168 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
