[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