This is an archive of the discontinued LLVM Phabricator instance.

[NFC] unsigned->Register in storeRegTo/loadRegFromStack
ClosedPublic

Authored by simoll on Feb 3 2020, 2:36 AM.

Details

Summary

This patch makes progress on the 'unsigned -> Register' rewrite for
TargetInstrInfo::loadRegFromStack and TII::storeRegToStack.

Diff Detail

Event Timeline

simoll created this revision.Feb 3 2020, 2:36 AM

Unit tests: pass. 62405 tests passed, 0 failed and 839 were skipped.

clang-tidy: fail. clang-tidy found 4 errors and 28 warnings. 0 of them are added as review comments below (why?).

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

arsenm accepted this revision.Feb 3 2020, 5:15 AM
This revision is now accepted and ready to land.Feb 3 2020, 5:15 AM
This revision was automatically updated to reflect the committed changes.