[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-26 Thread yanming via Phabricator via cfe-commits
ym1813382441 added inline comments. Comment at: llvm/include/llvm/Support/RISCVISAInfo.h:24 + unsigned Minor; + bool operator==(const RISCVExtensionVersion &Vers) const { +return this->Major == Vers.Major && this->Minor == Vers.Minor; eopXD wrote: > ym18133

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-26 Thread yanming via Phabricator via cfe-commits
ym1813382441 abandoned this revision. ym1813382441 added a comment. RISC-V International seem to have indicated they'd add new named extensions rather than add instructions to an existing extension in the future. Related discussions at D115921 Repository: rG

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-23 Thread yanming via Phabricator via cfe-commits
ym1813382441 created this revision. Herald added subscribers: luke, VincentWu, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, s

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-23 Thread yanming via Phabricator via cfe-commits
ym1813382441 added a comment. I want to use arrays to maintain the supported version number of each extension, this patch in using the `find` algorithm helps to simplify the code. Comment at: llvm/include/llvm/Support/RISCVISAInfo.h:24 + unsigned Minor; + bool operator==(con

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-23 Thread yanming via Phabricator via cfe-commits
ym1813382441 marked 2 inline comments as done. ym1813382441 added a comment. F26630797: Screenshot from 2023-02-24 15-54-21.png Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144696/new/ https://reviews.llvm.org

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-24 Thread yanming via Phabricator via cfe-commits
ym1813382441 added a comment. In D144696#4149516 , @craig.topper wrote: > Can you outline your full plan and why you want to do this. We need to see > the bigger picture. In D144696#4149545 , @ym1813382441 w

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-24 Thread yanming via Phabricator via cfe-commits
ym1813382441 added a comment. In D144696#4149560 , @craig.topper wrote: > In D144696#4149545 , @ym1813382441 > wrote: > >> F26630797: Screenshot from 2023-02-24 15-54-21.png >>

[PATCH] D144696: [RISCV][NFC] Package version number information using RISCVExtensionVersion.

2023-02-24 Thread yanming via Phabricator via cfe-commits
ym1813382441 added a comment. In D144696#4149706 , @craig.topper wrote: > How much of the existing vector code in the backend including tablegen can be > reused for 0.71? How much of the intrinsic interface can be used? If we’re > talking about complet