chill added inline comments.
================ Comment at: lib/CodeGen/TargetInfo.cpp:5055 + Alignment = getContext().getTypeNaturalAlign(Ty); + Alignment = std::min(std::max(Alignment, 64u), 128u); + } else { ---------------- t.p.northover wrote: > I think the max/min logic is more confusing here than the alternative: > > Alignment = Alignment < 128 ? 64 : 128; I'll change it. https://reviews.llvm.org/D46013 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits