[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