Page MenuHomePhabricator

[llvm-exegesis] Do not try to assign random registers twice.
ClosedPublic

Authored by courbet on Oct 29 2020, 3:56 AM.

Details

Summary

Doing a random assignment assigns both tested (forward) and back-to-back
(backward) instructions.

When none of the tested instruction and back-to-back instruction have
implicit aliasing, we're currently trying to do a random register
asignment twice.

Fix this (see PR26418).

Diff Detail

Event Timeline

courbet created this revision.Oct 29 2020, 3:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 29 2020, 3:56 AM
Herald added a subscriber: mstojanovic. · View Herald Transcript
courbet requested review of this revision.Oct 29 2020, 3:56 AM
gchatelet accepted this revision.Oct 29 2020, 5:22 AM

Good catch! Thx a lot.

This revision is now accepted and ready to land.Oct 29 2020, 5:22 AM
This revision was landed with ongoing or failed builds.Oct 29 2020, 5:27 AM
This revision was automatically updated to reflect the committed changes.