[PATCH] D32015: Make globalaa-retained.ll test catching more cases.

[Kristof Beyls via Phabricator via llvm-commits]

kristof.beyls added a comment.

In https://reviews.llvm.org/D32015#727113, @a.elovikov wrote:

> I have to note that after http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20170410/444886.html
>  [llvm] r300200 - Re-apply "[GVNHoist] Move GVNHoist to function simplification part of pipeline."
>  that test starts failing.


Does this mean that the test (if it had already been committed), would have found that r300200 breaks retaining GlobalsAA info; or is something wrong with the test?
Should GVNHoist be marked as preserving GlobalsAA? Or does it break GlobalsAA?


https://reviews.llvm.org/D32015