[clang] [llvm] [RISCV] Add B extension (PR #76893)

Pengcheng Wang via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 5 01:57:35 PDT 2024


================
@@ -920,8 +920,8 @@ void RISCVISAInfo::updateImplication() {
 }
 
 static constexpr StringLiteral CombineIntoExts[] = {
-    {"zk"},    {"zkn"},  {"zks"},   {"zvkn"},  {"zvknc"},
-    {"zvkng"}, {"zvks"}, {"zvksc"}, {"zvksg"},
+    {"b"},     {"zk"},    {"zkn"},  {"zks"},   {"zvkn"},
----------------
wangpc-pp wrote:

> Remove this from `CombineIntoExts` may cause `__riscv_b` become less useful I think?

Yeah, but I think @topperc's concern makes sense as well. Will binutils complain or just ignore unknown extensions?

https://github.com/llvm/llvm-project/pull/76893


More information about the cfe-commits mailing list