[clang] [NFC] Remove unnecessary copy of Triple in ItaniumMangle.cpp (PR #102469)
Abhina Sree via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 8 07:06:02 PDT 2024
https://github.com/abhina-sree updated https://github.com/llvm/llvm-project/pull/102469
>From e5139d8079471ac6e6f0d76f36ed7ef593fff1b7 Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan <Abhina.Sreeskantharajan at ibm.com>
Date: Thu, 8 Aug 2024 09:57:03 -0400
Subject: [PATCH 1/2] [NFC] Remove unnecessary copy of Triple in
ItaniumMangle.cpp
---
clang/lib/AST/ItaniumMangle.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/clang/lib/AST/ItaniumMangle.cpp b/clang/lib/AST/ItaniumMangle.cpp
index dc317f2a7870b..976670d1efa56 100644
--- a/clang/lib/AST/ItaniumMangle.cpp
+++ b/clang/lib/AST/ItaniumMangle.cpp
@@ -2727,8 +2727,6 @@ void CXXNameMangler::mangleQualifiers(Qualifiers Quals, const DependentAddressSp
// <type> ::= U <OpenCL-addrspace>
// <type> ::= U <CUDA-addrspace>
- llvm::Triple Triple = getASTContext().getTargetInfo().getTriple();
-
SmallString<64> ASString;
LangAS AS = Quals.getAddressSpace();
@@ -2800,7 +2798,7 @@ void CXXNameMangler::mangleQualifiers(Qualifiers Quals, const DependentAddressSp
// For z/OS, there are no special mangling rules applied to the ptr32
// qualifier. Ex: void foo(int * __ptr32 p) -> _Z3f2Pi. The mangling for
// "p" is treated the same as a regular integer pointer.
- if (!Triple.isOSzOS())
+ if (!getASTContext().getTargetInfo().getTriple().isOSzOS())
ASString = "ptr32_uptr";
break;
case LangAS::ptr64:
>From f488d931227fb0f19d5237de7a68942680c66a0b Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan <Abhina.Sreeskantharajan at ibm.com>
Date: Thu, 8 Aug 2024 10:05:50 -0400
Subject: [PATCH 2/2] fix another use as well
---
clang/lib/Sema/SemaType.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/clang/lib/Sema/SemaType.cpp b/clang/lib/Sema/SemaType.cpp
index 91c8f18648fa4..08020f9f88927 100644
--- a/clang/lib/Sema/SemaType.cpp
+++ b/clang/lib/Sema/SemaType.cpp
@@ -7057,7 +7057,6 @@ static bool handleMSPointerTypeQualifierAttr(TypeProcessingState &State,
// Add address space to type based on its attributes.
LangAS ASIdx = LangAS::Default;
- llvm::Triple Triple = S.Context.getTargetInfo().getTriple();
uint64_t PtrWidth =
S.Context.getTargetInfo().getPointerWidth(LangAS::Default);
if (PtrWidth == 32) {
@@ -7066,7 +7065,7 @@ static bool handleMSPointerTypeQualifierAttr(TypeProcessingState &State,
else if (Attrs[attr::UPtr])
ASIdx = LangAS::ptr32_uptr;
} else if (PtrWidth == 64 && Attrs[attr::Ptr32]) {
- if (Triple.isOSzOS() || Attrs[attr::UPtr])
+ if (S.Context.getTargetInfo().getTriple().isOSzOS() || Attrs[attr::UPtr])
ASIdx = LangAS::ptr32_uptr;
else
ASIdx = LangAS::ptr32_sptr;
More information about the cfe-commits
mailing list