This is an archive of the discontinued LLVM Phabricator instance.

[SCEV] Fix wrong initialization of MaxNotTaken
ClosedPublic

Authored by mkazantsev on Nov 23 2022, 12:56 AM.

Details

Summary

I'm not sure why, but since this code was introduced back in 2019, this
variable used wrong value to initialize. Somehow it just worked, but
needs to be fixed.

Diff Detail

Event Timeline

mkazantsev created this revision.Nov 23 2022, 12:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 12:56 AM
mkazantsev requested review of this revision.Nov 23 2022, 12:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 12:56 AM
fhahn accepted this revision.Nov 23 2022, 9:01 AM

LGTM, thanks! Surprising that the compiler didn't warn about the unused argument?

This revision is now accepted and ready to land.Nov 23 2022, 9:01 AM

Yeah, that's amazing.

This revision was landed with ongoing or failed builds.Nov 24 2022, 1:14 AM
This revision was automatically updated to reflect the committed changes.