Page MenuHomePhabricator

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

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

Diff Detail

Event Timeline

jroelofs created this revision.Thu, Jul 23, 11:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Jul 23, 11:04 AM
fhahn added a comment.Fri, Jul 24, 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.Fri, Jul 24, 1:49 PM

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