https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
}
static constexpr StringLiteral CombineIntoExts[] = {
-{"zk"},{"zkn"}, {"zks"}, {"zvkn"}, {"zvknc"},
-{"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+{"b"}, {"zk"},{"zkn"}, {"zks"},
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
}
static constexpr StringLiteral CombineIntoExts[] = {
-{"zk"},{"zkn"}, {"zks"}, {"zvkn"}, {"zvknc"},
-{"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+{"b"}, {"zk"},{"zkn"}, {"zks"},
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
}
static constexpr StringLiteral CombineIntoExts[] = {
-{"zk"},{"zkn"}, {"zks"}, {"zvkn"}, {"zvknc"},
-{"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+{"b"}, {"zk"},{"zkn"}, {"zks"},
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From ac2c10240160783ebbd65332230c8c294fef2442 Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH] [RISCV] Add B extension
It seems that we have `B` extension again:
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
}
static constexpr StringLiteral CombineIntoExts[] = {
-{"zk"},{"zkn"}, {"zks"}, {"zvkn"}, {"zvknc"},
-{"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+{"b"}, {"zk"},{"zkn"}, {"zks"},
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
}
static constexpr StringLiteral CombineIntoExts[] = {
-{"zk"},{"zkn"}, {"zks"}, {"zvkn"}, {"zvknc"},
-{"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+{"b"}, {"zk"},{"zkn"}, {"zks"},
wangpc-pp wrote:
Ping. The B extension has been ratified and I hava rebased this PR.
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From a780efe3e297fe59c5521ea5387dda12905a4423 Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH] [RISCV] Add B extension
It seems that we have `B` extension again:
kito-cheng wrote:
Jeff told me it's still need wait TSC vote for the ratification, anyway it will
ratify this month.
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
asb wrote:
> It passed public review[1] and merged into riscv-isa-manual[2], so I think
> it's time to mark it as 1.0 and moving forward :)
>
> [1]
> https://groups.google.com/a/groups.riscv.org/g/isa-dev/c/KetVUCQkfK4/m/Y3Dbd2pvAAAJ?utm_medium=email_source=footer
> [2]
>
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From 169ef33f585b964d9af7a7628919271245f318fc Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH 1/3] [RISCV] Add B extension
It seems that we have `B` extension
kito-cheng wrote:
Could you add `B` into CombinedExtsEntry and added a test for that?
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From 169ef33f585b964d9af7a7628919271245f318fc Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH 1/2] [RISCV] Add B extension
It seems that we have `B` extension
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From 169ef33f585b964d9af7a7628919271245f318fc Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH] [RISCV] Add B extension
It seems that we have `B` extension again:
kito-cheng wrote:
It passed public review[1] and merged into riscv-isa-manual[2], so I think it's
time to mark it as 1.0 and moving forward :)
[1]
https://groups.google.com/a/groups.riscv.org/g/isa-dev/c/KetVUCQkfK4/m/Y3Dbd2pvAAAJ?utm_medium=email_source=footer
[2]
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/76893
>From c4f7441b2809dfbb6ed4b6d97d581e3d736819fe Mon Sep 17 00:00:00 2001
From: wangpc
Date: Thu, 4 Jan 2024 13:05:53 +0800
Subject: [PATCH] [RISCV] Add B extension
It seems that we have `B` extension again:
kito-cheng wrote:
It tagged with 0.1 now :)
https://github.com/riscv/riscv-b/releases/tag/v0.1
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
topperc wrote:
> Thanks!
>
>
>
> If someone sets zba_zbb_zbs, should b be inferred?
>
>
>
Yes, but it will break `-march=rv64i_zba_zbb_zbs -fno-integrated-as` with
versions of the external assembler that don't know about B. We'll infer B and
pass it to the assembler -march.
asb wrote:
Thanks!
If someone sets zba_zbb_zbs, should b be inferred?
This patch needs an LLVM release note as well, and agreed we should await on a
versioning decision.
https://github.com/llvm/llvm-project/pull/76893
___
cfe-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Wang Pengcheng (wangpc-pp)
Changes
It seems that we have `B` extension again: https://github.com/riscv/riscv-b
According to the spec, `B` extension represents the collection of
the `Zba`, `Zbb`, `Zbs` extensions.
Though it hasn't been
https://github.com/wangpc-pp created
https://github.com/llvm/llvm-project/pull/76893
It seems that we have `B` extension again: https://github.com/riscv/riscv-b
According to the spec, `B` extension represents the collection of
the `Zba`, `Zbb`, `Zbs` extensions.
Though it hasn't been
22 matches
Mail list logo
