Skip to content

Commit 43d7520

Browse files
al45taircompnerd
andcommitted
[RuntimeDyld][Windows] Eliminate a local variable.
No need for `TargetName` to exist separately, really. Co-authored-by: Saleem Abdulrasool <[email protected]>
1 parent 1c27fe0 commit 43d7520

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldCOFF.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,15 +118,14 @@ bool RuntimeDyldCOFF::isCompatibleFile(const object::ObjectFile &Obj) const {
118118
return Obj.isCOFF();
119119
}
120120

121-
bool RuntimeDyldCOFF::relocationNeedsDLLImportStub(const RelocationRef &R) const {
121+
bool RuntimeDyldCOFF::relocationNeedsDLLImportStub(
122+
const RelocationRef &R) const {
122123
object::symbol_iterator Symbol = R.getSymbol();
123124
Expected<StringRef> TargetNameOrErr = Symbol->getName();
124125
if (!TargetNameOrErr)
125126
return false;
126127

127-
StringRef TargetName = *TargetNameOrErr;
128-
129-
return TargetName.startswith(getImportSymbolPrefix());
128+
return TargetNameOrErr->startswith(getImportSymbolPrefix());
130129
}
131130

132131
} // namespace llvm

0 commit comments

Comments
 (0)