[PATCH] Take alignment into account in isSafeToLoadUnconditionally
Artur Pilipenko
apilipenko at azulsystems.com
Tue Jun 16 10:50:41 PDT 2015
================
Comment at: lib/Analysis/Loads.cpp:69
@@ +68,3 @@
+
+ // Require ABI alignment for loads without alignment specification
+ if (Align == 0)
----------------
reames wrote:
> Please clarify comment as to *why* we want this.
That is a part of load specification:
"A value of 0 or an omitted align argument means that the operation has the ABI alignment for the target."
http://reviews.llvm.org/D10475
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list