This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][NFC] Use defvar to simplify pattern definations.
ClosedPublic

Authored by Pretty-box on Apr 15 2022, 12:36 AM.

Diff Detail

Event Timeline

Pretty-box created this revision.Apr 15 2022, 12:36 AM
Pretty-box requested review of this revision.Apr 15 2022, 12:36 AM
jacquesguan added inline comments.Apr 15 2022, 1:01 AM
llvm/lib/Target/RISCV/RISCVInstrInfoVVLPatterns.td
707

Can these two lines be merged into one line if is less than 80 cols?

829

Should adjust the align here.

Pretty-box marked 2 inline comments as done.
This revision is now accepted and ready to land.Apr 15 2022, 2:17 AM

Address comment

frasercrmck accepted this revision.Apr 18 2022, 12:19 AM

LGTM too

This revision was landed with ongoing or failed builds.Apr 21 2022, 7:45 PM
This revision was automatically updated to reflect the committed changes.