[PATCH] D144384: Bump RV32E version to 2.0

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 20 05:26:50 PST 2023


asb added a comment.

Although a trivial change in terms of lines edited, I suggested Job posted this separately from the existing D143570 <https://reviews.llvm.org/D143570> review on the basis that it might trigger unrelated concerns.

I think it makes sense to go ahead and bump this version. The strict error checking in tools is being addressed, and given how limited our RVE support is in LLVM (no codegen supported), I'd be surprised if there are any downstream users right now. The reason D143570 <https://reviews.llvm.org/D143570> triggers this is that technically RV64E 1.9 didn't exist, so it seems cleaner to move to the ratified version number.

This LGTM, but I'm of course waiting for a bit in case there are other viewpoints.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144384/new/

https://reviews.llvm.org/D144384



More information about the llvm-commits mailing list