[clang] [clang][RISCV] Extend intrinsic size check variable from 16 -> 32 bits. NFC (PR #111481)

Brandon Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 7 22:05:00 PDT 2024


https://github.com/4vtomat created https://github.com/llvm/llvm-project/pull/111481

We currently have over 67000 intrinsics, uint16_t will overflow.


>From f6a27f7de2459e22961555054b92a61a0e7ededf Mon Sep 17 00:00:00 2001
From: Brandon Wu <brandon.wu at sifive.com>
Date: Mon, 7 Oct 2024 22:03:43 -0700
Subject: [PATCH] [clang][RISCV] Extend intrinsic size check variable from 16
 -> 32 bits. NFC

We currently have over 67000 intrinsics, uint16_t will overflow.
---
 clang/lib/Sema/SemaRISCV.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/clang/lib/Sema/SemaRISCV.cpp b/clang/lib/Sema/SemaRISCV.cpp
index 3da4b515b1b114..f49c2088ef4776 100644
--- a/clang/lib/Sema/SemaRISCV.cpp
+++ b/clang/lib/Sema/SemaRISCV.cpp
@@ -399,7 +399,7 @@ void RISCVIntrinsicManagerImpl::InitRVVIntrinsic(
                                      Record.HasFRMRoundModeOp);
 
   // Put into IntrinsicList.
-  uint16_t Index = IntrinsicList.size();
+  uint32_t Index = IntrinsicList.size();
   assert(IntrinsicList.size() == (size_t)Index &&
          "Intrinsics indices overflow.");
   IntrinsicList.push_back({BuiltinName, Signature});



More information about the cfe-commits mailing list