[PATCH] D56568: [AliasSetTracker] Store AliasResult and pass it on mergeSetIn.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea marked an inline comment as done.
asbirlea added inline comments.


================
Comment at: lib/Analysis/AliasSetTracker.cpp:299
     if (!FoundSet) {      // If this is the first alias set ptr can go into.
       FoundSet = &*Cur;   // Remember it.
     } else {              // Otherwise, we must merge the sets.
----------------
I see your point above, thanks for noticing!
I believe the issue is resolved over here with adding:
```
if (AR != MustAlias)
   FoundSet.Alias = SetMayAlias;
```
What do you think?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56568/new/

https://reviews.llvm.org/D56568