[PATCH] D20658: MemorySSA: Revert r269678 and r268068; replace with special casing in MemorySSA.
Daniel Berlin via llvm-commits
llvm-commits at lists.llvm.org
Wed May 25 20:22:18 PDT 2016
LGTM
On Wed, May 25, 2016 at 6:52 PM, Peter Collingbourne <peter at pcc.me.uk>
wrote:
> pcc created this revision.
> pcc added reviewers: dberlin, chandlerc, hfinkel, reames, sanjoy, boazo,
> majnemer, gberry.
> pcc added a subscriber: llvm-commits.
>
> It turns out that too many passes are relying on alias analysis results
> for control dependencies. Until we fix that by introducing a more accurate
> modelling of control dependencies, special case assume in MemorySSA
> instead.
>
> Also introduce tests to ensure we don't regress the FunctionAttrs or LICM
> passes.
>
> http://reviews.llvm.org/D20658
>
> Files:
> lib/Analysis/BasicAliasAnalysis.cpp
> lib/Transforms/Utils/MemorySSA.cpp
> test/Transforms/FunctionAttrs/assume.ll
> test/Transforms/LICM/assume.ll
> test/Transforms/Util/MemorySSA/assume.ll
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160525/12321858/attachment.html>
More information about the llvm-commits
mailing list