summaryrefslogtreecommitdiffstats
path: root/clang/lib/Basic
diff options
context:
space:
mode:
authorCraig Topper <craig.topper@gmail.com>2015-10-21 04:52:36 +0000
committerCraig Topper <craig.topper@gmail.com>2015-10-21 04:52:36 +0000
commit67288a9b752e78d61d35267ea7be79ea385ad94b (patch)
treec68c842a8bd872baecd9bdfb600dd7ca14e60f21 /clang/lib/Basic
parentde330c78b3c1a960009fc962758e0bce88c5339b (diff)
downloadbcm5719-llvm-67288a9b752e78d61d35267ea7be79ea385ad94b.tar.gz
bcm5719-llvm-67288a9b752e78d61d35267ea7be79ea385ad94b.zip
Parse into an unsigned type instead of a signed type and then checking for positive and casting to unsigned. Since we know the string starts with a digit it couldn't be negative anyway. NFCI
llvm-svn: 250879
Diffstat (limited to 'clang/lib/Basic')
-rw-r--r--clang/lib/Basic/TargetInfo.cpp9
1 files changed, 4 insertions, 5 deletions
diff --git a/clang/lib/Basic/TargetInfo.cpp b/clang/lib/Basic/TargetInfo.cpp
index 35044953810..048a6d64ba8 100644
--- a/clang/lib/Basic/TargetInfo.cpp
+++ b/clang/lib/Basic/TargetInfo.cpp
@@ -358,9 +358,9 @@ bool TargetInfo::isValidGCCRegisterName(StringRef Name) const {
// If we have a number it maps to an entry in the register name array.
if (isDigit(Name[0])) {
- int n;
+ unsigned n;
if (!Name.getAsInteger(0, n))
- return n >= 0 && (unsigned)n < Names.size();
+ return n < Names.size();
}
// Check register names.
@@ -406,10 +406,9 @@ TargetInfo::getNormalizedGCCRegisterName(StringRef Name) const {
// First, check if we have a number.
if (isDigit(Name[0])) {
- int n;
+ unsigned n;
if (!Name.getAsInteger(0, n)) {
- assert(n >= 0 && (unsigned)n < Names.size() &&
- "Out of bounds register number!");
+ assert(n < Names.size() && "Out of bounds register number!");
return Names[n];
}
}
OpenPOWER on IntegriCloud