This is an archive of the discontinued LLVM Phabricator instance.

[test] Add -verify-coalescing to testcase and fix problems with it
ClosedPublic

Authored by uabelho on Aug 21 2023, 1:27 AM.

Details

Summary

Apparently the testcase
coalesce-partial-redundant-reguse-terminator.mir
was broken in a way that -verify-coalescing detected.

Update the testcase so -verify-coalescing doesn't complain and so
that it still exposes the problem originally fixed in 6c062b7641623.

Diff Detail

Event Timeline

uabelho created this revision.Aug 21 2023, 1:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2023, 1:27 AM
uabelho requested review of this revision.Aug 21 2023, 1:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2023, 1:27 AM
uabelho updated this revision to Diff 551929.Aug 21 2023, 1:37 AM

Fixed CHECK messup in first patch.

arsenm accepted this revision.Aug 21 2023, 2:34 PM

Thanks

This revision is now accepted and ready to land.Aug 21 2023, 2:34 PM