[PATCH] D146451: [RISCV] Replace RISCV->RISC-V in strings.

2023-03-27 Thread Craig Topper via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0f4c9c016caf: [RISCV] Replace RISCV->RISC-V in strings. (authored by craig.topper). Changed prior to commit: https://reviews.llvm.org/D146451?vs=5

[PATCH] D146451: [RISCV] Replace RISCV->RISC-V in strings.

2023-03-27 Thread Alex Bradbury via Phabricator via cfe-commits
asb accepted this revision. asb added a comment. This revision is now accepted and ready to land. Not as compelling as user facing strings or comments that clearly refer to the RISC-V ISA rather than the "RISCV" LLVM backend. But I don't really see a good reason not to be consistent. LGTM. Rep

[PATCH] D146451: [RISCV] Replace RISCV->RISC-V in strings.

2023-03-20 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D146451#4207261 , @asb wrote: > The logic I saw with the pass names is that the backend for the RISC-V > architecture is called "RISCV". But I've got no objection to changing. Yeah. I also wasn't sure which is best. Re

[PATCH] D146451: [RISCV] Replace RISCV->RISC-V in strings.

2023-03-20 Thread Alex Bradbury via Phabricator via cfe-commits
asb added a comment. The logic I saw with the pass names is that the backend for the RISC-V architecture is called "RISCV". But I've got no objection to changing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146451/new/ https://reviews.llvm.org/D

[PATCH] D146451: [RISCV] Replace RISCV->RISC-V in strings.

2023-03-20 Thread Craig Topper via Phabricator via cfe-commits
craig.topper created this revision. craig.topper added reviewers: reames, asb, jrtc27, kito-cheng, luismarques. Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, evandro, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck