[PATCH] D50730: [AST] Generalize argument specific aliasing

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 14 20:14:08 PDT 2018


mkazantsev added a comment.

I am concerned that these changes didn't change anything in behavior of dedicated tests in `test/Analysis/AliasSet`. I think we should commit some of those as NFC and then show how this patch changes behavior of these tests.



================
Comment at: lib/Analysis/AliasSetTracker.cpp:443
+    }
+    return;
+  }  
----------------
Aren't we supposed to return some value here?


Repository:
  rL LLVM

https://reviews.llvm.org/D50730





More information about the llvm-commits mailing list