This is an archive of the discontinued LLVM Phabricator instance.

[CodeExtractor]: Fix an extractor bug of missing replacing use of old phi with the new one
ClosedPublic

Authored by davidxl on Apr 24 2017, 8:35 PM.

Details

Summary

This is triggered by partial inliner improvement patch (so a test case is not yet available) -- but the bug seems obvious. (found with clang built with partial inlining turned on -- it causes infinite loop during indirect call promotion).

Diff Detail

Event Timeline

davidxl created this revision.Apr 24 2017, 8:35 PM
davide accepted this revision.Apr 24 2017, 8:49 PM

uh, how this ever worked is clearly beyond me.
If you can add these tests to the inliner improvement patch, that would be great.

This revision is now accepted and ready to land.Apr 24 2017, 8:49 PM
This revision was automatically updated to reflect the committed changes.