[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