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

Reply via email to