This is an archive of the discontinued LLVM Phabricator instance.

Fixup test to accept different Opcode values
ClosedPublic

Authored by dyung on Jan 9 2017, 2:49 PM.

Details

Summary

The test is searching for a specific opcode when it doesn't really need to. This causes a problem when the opcode changes for whatever reason, so update the test with a change similar to r274774 to make the test more robust.

Diff Detail

Event Timeline

dyung updated this revision to Diff 83708.Jan 9 2017, 2:49 PM
dyung retitled this revision from to Fixup test to accept different Opcode values.
dyung updated this object.
dyung added reviewers: t.p.northover, sanjoy.
dyung added a subscriber: llvm-commits.

LGTM - I've got the same change as part of https://reviews.llvm.org/D28079 - that part already got approved there.
Feel free to commit.

This revision was automatically updated to reflect the committed changes.
dyung added a comment.Jan 10 2017, 2:27 PM

LGTM - I've got the same change as part of https://reviews.llvm.org/D28079 - that part already got approved there.
Feel free to commit.

Thanks! I've committed the change in r291605.