[PATCH] Take alignment into account for speculative loads
Artur Pilipenko
apilipenko at azulsystems.com
Fri May 15 05:41:49 PDT 2015
Hi hfinkel, reames, sanjoy,
Take alignment into account in isSafeToSpeculativelyExecute and isSafeToLoadUnconditionally.
I assume that pointers without explicit alignment have ABI alignment. I'm not sure that it's correct but for now it reduces the impact of the change. Otherwise we can't speculate on pointers without explicit alignment because loads without explicit alignment require ABI aligned pointers.
As a next step I'm going to common isDereferenceableAndAlignedPointer and isSafeToLoadUnconditionally code.
http://reviews.llvm.org/D9791
Files:
include/llvm/Analysis/ValueTracking.h
lib/Analysis/Loads.cpp
lib/Analysis/ValueTracking.cpp
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9791.25858.patch
Type: text/x-patch
Size: 12639 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150515/c3c9dd19/attachment.bin>
More information about the llvm-commits
mailing list