[clang] [Clang][TableGen] Use const pointers for various `Init *` pointers (PR #112321)
Rahul Joshi via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 15 08:15:04 PDT 2024
https://github.com/jurahul updated https://github.com/llvm/llvm-project/pull/112321
>From 4743134d6b6ef21fc6b09dea59c30d6be31ef3fd Mon Sep 17 00:00:00 2001
From: Rahul Joshi <rjoshi at nvidia.com>
Date: Mon, 14 Oct 2024 23:01:19 -0700
Subject: [PATCH] [Clang][TableGen] Use const pointers for various `Init *`
pointers
---
clang/utils/TableGen/ClangSACheckersEmitter.cpp | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/clang/utils/TableGen/ClangSACheckersEmitter.cpp b/clang/utils/TableGen/ClangSACheckersEmitter.cpp
index bebdcac3212613..36012dbf70791b 100644
--- a/clang/utils/TableGen/ClangSACheckersEmitter.cpp
+++ b/clang/utils/TableGen/ClangSACheckersEmitter.cpp
@@ -29,7 +29,7 @@ static std::string getPackageFullName(const Record *R, StringRef Sep = ".");
static std::string getParentPackageFullName(const Record *R,
StringRef Sep = ".") {
std::string name;
- if (DefInit *DI = dyn_cast<DefInit>(R->getValueInit("ParentPackage")))
+ if (const DefInit *DI = dyn_cast<DefInit>(R->getValueInit("ParentPackage")))
name = getPackageFullName(DI->getDef(), Sep);
return name;
}
@@ -53,7 +53,7 @@ static std::string getCheckerFullName(const Record *R, StringRef Sep = ".") {
}
static std::string getStringValue(const Record &R, StringRef field) {
- if (StringInit *SI = dyn_cast<StringInit>(R.getValueInit(field)))
+ if (const StringInit *SI = dyn_cast<StringInit>(R.getValueInit(field)))
return std::string(SI->getValue());
return std::string();
}
@@ -94,7 +94,7 @@ static std::string getCheckerDocs(const Record &R) {
/// the class itself has to be modified for adding a new option type in
/// CheckerBase.td.
static std::string getCheckerOptionType(const Record &R) {
- if (BitsInit *BI = R.getValueAsBitsInit("Type")) {
+ if (const BitsInit *BI = R.getValueAsBitsInit("Type")) {
switch(getValueFromBitsInit(BI, R)) {
case 0:
return "int";
@@ -111,7 +111,7 @@ static std::string getCheckerOptionType(const Record &R) {
}
static std::string getDevelopmentStage(const Record &R) {
- if (BitsInit *BI = R.getValueAsBitsInit("DevelopmentStage")) {
+ if (const BitsInit *BI = R.getValueAsBitsInit("DevelopmentStage")) {
switch(getValueFromBitsInit(BI, R)) {
case 0:
return "alpha";
@@ -131,7 +131,7 @@ static bool isHidden(const Record *R) {
return true;
// Not declared as hidden, check the parent package if it is hidden.
- if (DefInit *DI = dyn_cast<DefInit>(R->getValueInit("ParentPackage")))
+ if (const DefInit *DI = dyn_cast<DefInit>(R->getValueInit("ParentPackage")))
return isHidden(DI->getDef());
return false;
More information about the cfe-commits
mailing list