[PATCH] D12065: [WinEHPrepare] Fix catchret successor phi demotion

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 16 12:50:50 PDT 2015


majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.

Nice catch, LGTM.

As an aside, would it be more optimal (and legal) to have a single slot for the value contained within `%phi`?  We'd still need two stores but I think we could save a `load`.  If so, we might want a TODO for it if we don't already have an appropriate one elsewhere.


http://reviews.llvm.org/D12065





More information about the llvm-commits mailing list