rsmith added a comment. In https://reviews.llvm.org/D28166#633621, @aaron.ballman wrote:
> Do you think this patch should be gated on (or perhaps combined with) a fix > for the lowering bug, or do you think this patch is reasonable on its own? > Given that it turns working code into UB, I think my preference is to gate it > on a fix for the lowering bug, but I'm also not certain I am the right person > to implement that fix (though I could give it a shot). The test in question has a comment pointing to PR7117, which in turn indicates that we might miscompile parts of FreeBSD if we land this as-is. So I think we need to gate this on a fix for the lowering bug. https://reviews.llvm.org/D28166 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
