This is an archive of the discontinued LLVM Phabricator instance.

Simplify/cleanup BasicBlockUtilsTest
ClosedPublic

Authored by MatzeB on Feb 17 2022, 3:46 PM.

Details

Summary

Cleanup BasicBolckUtilsTest using C++ raw string literals, remove duplicated block functions and smaller style changes.

Diff Detail

Event Timeline

MatzeB created this revision.Feb 17 2022, 3:46 PM
MatzeB requested review of this revision.Feb 17 2022, 3:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 17 2022, 3:46 PM

The Windows build testing is failing. Took a look and pretty sure you just need to rebase past 19bdf44d850884a13a8708ccf1260fb7f4ef4eb3 which reverted the patch with the failing tests.

MatzeB updated this revision to Diff 410878.Feb 23 2022, 11:21 AM
modimo accepted this revision.Feb 23 2022, 1:12 PM

LGTM

This revision is now accepted and ready to land.Feb 23 2022, 1:12 PM
wenlei accepted this revision.Feb 23 2022, 5:53 PM