[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