Page MenuHomePhabricator

[LateJumpThreading] Enable LateJumpThreading right before CGP.
AbandonedPublic

Authored by bmakam on Sep 14 2017, 2:08 PM.

Details

Reviewers
None

Diff Detail

Event Timeline

bmakam created this revision.Sep 14 2017, 2:08 PM
kparzysz added inline comments.
lib/CodeGen/TargetPassConfig.cpp
669

That should be under "opt level != none".

bmakam updated this revision to Diff 115288.Sep 14 2017, 2:30 PM

Address Krzysztof's comment.

bmakam marked an inline comment as done.Sep 14 2017, 2:30 PM

What's going on with this patch? We also saw a performance regression on one of our benchmarks. And I was interested in whether this woudl help.

What's going on with this patch? We also saw a performance regression on one of our benchmarks. And I was interested in whether this woudl help.

Have you tested it? Could you share your results if it fixed the regression? I have only tested it on the benchmark that regressed for us and it recovered the regression, but need to test some more.

bmakam abandoned this revision.Feb 2 2018, 11:23 AM

D42260 seems a better fix to this regression.