This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Replaced !subst chain with !foldl
ClosedPublic

Authored by nitinjohnraj on Feb 21 2023, 6:43 PM.

Diff Detail

Event Timeline

nitinjohnraj created this revision.Feb 21 2023, 6:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 21 2023, 6:43 PM
nitinjohnraj requested review of this revision.Feb 21 2023, 6:43 PM
craig.topper retitled this revision from [RISCV] Replaced \!subst chain with \!foldl to [RISCV] Replaced !subst chain with !foldl.
craig.topper added inline comments.Feb 21 2023, 7:17 PM
llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td
51

For everyone else, I discussed this with Nitin internally. The list here is in reverse order because the original list was applied from the inside out. So "Pseudo" was substituted first. The order almost doesn't matter except that "_B16" must be replaced before "_B1" since "_B1" is a prefix of "_B16"

This revision is now accepted and ready to land.Feb 22 2023, 8:49 AM
This revision was automatically updated to reflect the committed changes.