[PATCH] D30903: [AArch64] Use alias analysis in the load/store optimization pass.

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 14 09:55:31 PDT 2017


mcrosier added a comment.

In https://reviews.llvm.org/D30903#699715, @MatzeB wrote:

> Looks good to me. Could you do the sanity checking that we do the right thing in CodeGen and compute the AliasAnalysis information only once for all of CodeGen and not repeat it for different passes? (i.e. -debug-pass=Executions should only show them computed once for all CodeGen passes). Feel free to delegate this task to https://reviews.llvm.org/D30839 if you want ;-)


Sure, Matthias.  After this change the Function AA result is freed after the AArch64 load/store optimization pass, rather than the Machine LICM pass (i.e., the change only extends the lifetime of the AA info and doesn't not require it to be recomputed).


https://reviews.llvm.org/D30903





More information about the llvm-commits mailing list