[llvm] [llvm] Use StringRef::consume_front (NFC) (PR #139458)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Sun May 11 09:43:50 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/139458
None
>From 72e27aa1b735987bb6e5f25523b7aa179e4eaed9 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sun, 11 May 2025 08:08:38 -0700
Subject: [PATCH] [llvm] Use StringRef::consume_front (NFC)
---
llvm/lib/Support/APFloat.cpp | 10 +++-------
llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp | 4 +---
2 files changed, 4 insertions(+), 10 deletions(-)
diff --git a/llvm/lib/Support/APFloat.cpp b/llvm/lib/Support/APFloat.cpp
index a7b9f259bfed5..e99649d26ce88 100644
--- a/llvm/lib/Support/APFloat.cpp
+++ b/llvm/lib/Support/APFloat.cpp
@@ -3262,9 +3262,8 @@ bool IEEEFloat::convertFromStringSpecials(StringRef str) {
return true;
}
- bool IsNegative = str.front() == '-';
+ bool IsNegative = str.consume_front("-");
if (IsNegative) {
- str = str.drop_front();
if (str.size() < MIN_NAME_SIZE)
return false;
@@ -3275,16 +3274,13 @@ bool IEEEFloat::convertFromStringSpecials(StringRef str) {
}
// If we have a 's' (or 'S') prefix, then this is a Signaling NaN.
- bool IsSignaling = str.front() == 's' || str.front() == 'S';
+ bool IsSignaling = str.consume_front_insensitive("s");
if (IsSignaling) {
- str = str.drop_front();
if (str.size() < MIN_NAME_SIZE)
return false;
}
- if (str.starts_with("nan") || str.starts_with("NaN")) {
- str = str.drop_front(3);
-
+ if (str.consume_front("nan") || str.consume_front("NaN")) {
// A NaN without payload.
if (str.empty()) {
makeNaN(IsSignaling, IsNegative);
diff --git a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
index ad42c73e24333..251828b6bc35b 100644
--- a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
@@ -1632,10 +1632,8 @@ SPIRVType *SPIRVGlobalRegistry::getOrCreateSPIRVTypeByName(
auto SpirvTy = getOrCreateSPIRVType(Ty, MIRBuilder, AQ, false, true);
// Handle "type*" or "type* vector[N]".
- if (TypeStr.starts_with("*")) {
+ if (TypeStr.consume_front("*"))
SpirvTy = getOrCreateSPIRVPointerType(Ty, MIRBuilder, SC);
- TypeStr = TypeStr.substr(strlen("*"));
- }
// Handle "typeN*" or "type vector[N]*".
bool IsPtrToVec = TypeStr.consume_back("*");
More information about the llvm-commits
mailing list