Page MenuHomePhabricator

Testcase for "More conservatively report status from LoopIdiomRecognize"
Needs ReviewPublic

Authored by jroelofs on Jul 23 2020, 11:04 AM.

Diff Detail

Event Timeline

jroelofs created this revision.Jul 23 2020, 11:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 23 2020, 11:04 AM
fhahn added a comment.Jul 24 2020, 1:42 PM

Thanks for sharing. I did not manage to get it to crash with the patches removed though :(

Thanks for sharing. I did not manage to get it to crash with the patches removed though :(

This must be super sensitive to something in the in-memory representation, since it doesn't even repro if I dump the IR and pass it back to the pass in question.

nikic added a comment.Jul 24 2020, 1:49 PM

Can you check whether it reproduces if you export the IR with --preserve-bc-uselistorder / --preserve-ll-uselistorder?

nikic resigned from this revision.Sep 3 2020, 1:12 PM