Skip to content

Commit 12234f8

Browse files
committed
Use StringRef::contains().
llvm-svn: 308526
1 parent 1985e9b commit 12234f8

File tree

5 files changed

+11
-13
lines changed

5 files changed

+11
-13
lines changed

lld/COFF/Driver.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ StringRef LinkerDriver::doFindFile(StringRef Filename) {
247247
bool HasPathSep = (Filename.find_first_of("/\\") != StringRef::npos);
248248
if (HasPathSep)
249249
return Filename;
250-
bool HasExt = (Filename.find('.') != StringRef::npos);
250+
bool HasExt = Filename.contains('.');
251251
for (StringRef Dir : SearchPaths) {
252252
SmallString<128> Path = Dir;
253253
sys::path::append(Path, Filename);
@@ -275,7 +275,7 @@ Optional<StringRef> LinkerDriver::findFile(StringRef Filename) {
275275
// Find library file from search path.
276276
StringRef LinkerDriver::doFindLib(StringRef Filename) {
277277
// Add ".lib" to Filename if that has no file extension.
278-
bool HasExt = (Filename.find('.') != StringRef::npos);
278+
bool HasExt = Filename.contains('.');
279279
if (!HasExt)
280280
Filename = Saver.save(Filename + ".lib");
281281
return doFindFile(Filename);

lld/COFF/DriverUtils.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -459,12 +459,12 @@ Export parseExport(StringRef Arg) {
459459
if (E.Name.empty())
460460
goto err;
461461

462-
if (E.Name.find('=') != StringRef::npos) {
462+
if (E.Name.contains('=')) {
463463
StringRef X, Y;
464464
std::tie(X, Y) = E.Name.split("=");
465465

466466
// If "<name>=<dllname>.<name>".
467-
if (Y.find(".") != StringRef::npos) {
467+
if (Y.contains(".")) {
468468
E.Name = X;
469469
E.ForwardTo = Y;
470470
return E;

lld/ELF/Error.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ static void newline(const Twine &Msg) {
4141

4242
if (Flag)
4343
*ErrorOS << "\n";
44-
Flag = (StringRef(Msg.str()).find('\n') != StringRef::npos);
44+
Flag = StringRef(Msg.str()).contains('\n');
4545
}
4646

4747
static void print(StringRef S, raw_ostream::Colors C) {

lld/ELF/SymbolTable.cpp

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -325,11 +325,9 @@ Symbol *SymbolTable<ELFT>::addUndefined(StringRef Name, bool IsLocal,
325325
// .symver foo,foo@@@VER
326326
// we can delete this hack.
327327
static int compareVersion(Symbol *S, StringRef Name) {
328-
if (Name.find("@@") != StringRef::npos &&
329-
S->body()->getName().find("@@") == StringRef::npos)
328+
if (Name.contains("@@") && !S->body()->getName().contains("@@"))
330329
return 1;
331-
if (Name.find("@@") == StringRef::npos &&
332-
S->body()->getName().find("@@") != StringRef::npos)
330+
if (!Name.contains("@@") && S->body()->getName().contains("@@"))
333331
return -1;
334332
return 0;
335333
}
@@ -720,7 +718,7 @@ void SymbolTable<ELFT>::assignExactVersion(SymbolVersion Ver,
720718
// Skip symbols containing version info because symbol versions
721719
// specified by symbol names take precedence over version scripts.
722720
// See parseSymbolVersion().
723-
if (B->getName().find('@') != StringRef::npos)
721+
if (B->getName().contains('@'))
724722
continue;
725723

726724
Symbol *Sym = B->symbol();

lld/lib/Core/Reproduce.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,9 @@ std::string lld::relativeToRoot(StringRef Path) {
4444

4545
// Quote a given string if it contains a space character.
4646
std::string lld::quote(StringRef S) {
47-
if (S.find(' ') == StringRef::npos)
48-
return S;
49-
return ("\"" + S + "\"").str();
47+
if (S.contains(' '))
48+
return ("\"" + S + "\"").str();
49+
return S;
5050
}
5151

5252
std::string lld::rewritePath(StringRef S) {

0 commit comments

Comments
 (0)