[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-12-09 Thread Yueh-Ting Chen via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe308b8e0c71b: [RISCV] Fix arch string parsing for multi-character extensions (authored by eopXD). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-12-09 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 393174. eopXD added a comment. One last rebase before landing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215 Files: clang/test/Driver/riscv-arch.c llvm/lib/Suppor

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-11-07 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 385329. eopXD added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215 Files: clang/test/Driver/riscv-arch.c llvm/lib/Support/RISCVISAInfo.cpp Ind

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-11-06 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. This seems to match the emerging consensus in the various RISC-V GitHub issue threads and as you point out, is needed to support extensions in the 1.0 V spec. Looks good to me - thanks! =

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-11-02 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD added a reviewer: asb. eopXD added a comment. @asb @jrtc27 Hi Alex and Jessica, Although the issue on relaxing naming rules aren't resolved in riscv-isa-manual [1], the v-spec v1.0 had come out including `zve` and `zvl` extensions and is frozen [2]. I have just finished a series of patch

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-11-01 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 383773. eopXD added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215 Files: clang/test/Driver/riscv-arch.c llvm/lib/Support/RISCVISAInfo.cpp Inde

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-11-01 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 383732. eopXD added a comment. rebase.` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215 Files: clang/test/Driver/riscv-arch.c llvm/lib/Support/RISCVISAInfo.cpp In

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-10-30 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 383605. eopXD added a comment. Herald added subscribers: VincentWu, luke957. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215 Files: clang/test/Driver/riscv-ar

[PATCH] D109215: [RISCV] Fix arch string parsing for multi-character extensions

2021-10-23 Thread Yueh-Ting Chen via Phabricator via cfe-commits
eopXD updated this revision to Diff 381738. eopXD added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fix test case fail. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109215/new/ https://reviews.llvm.org/D109215