[PATCH] D139025: [NFC][RISCV] Extract utility to calculate value through MajorVersion and MinorVersion
Alex Bradbury via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 30 10:50:18 PST 2022
asb added inline comments.
================
Comment at: llvm/include/llvm/Support/RISCVISAInfo.h:25
+ unsigned getVersionValue() {
+ return MajorVersion * 1000000 + MinorVersion * 1000;
+ }
----------------
Would this be better as `return getVersionValue(MajorVersion, MinorVersion);` in order to maximise logic reuse?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139025/new/
https://reviews.llvm.org/D139025
More information about the cfe-commits
mailing list