[PATCH] llvm.noalias - BasicAA should look through them too
Daniel Berlin
dberlin at dberlin.org
Thu Apr 30 08:56:11 PDT 2015
================
Comment at: include/llvm/IR/Value.h:404
@@ -405,1 +403,3 @@
+ const Value *stripPointerCasts(bool LookThroughNoAlias = false) const {
+ return const_cast<Value*>(this)->stripPointerCasts(LookThroughNoAlias);
}
----------------
So, is there actually a place you plan on not passing true here, or are you just trying to keep existing logic where you haven't explicitly modified it?
(Because i am having a bit of trouble seeing where you would want to pass false. At the point you are stripping, why is noalias more magical to not strip than anything else that can appear here?)
http://reviews.llvm.org/D9383
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list