forked from OSchip/llvm-project
Run clang-tidy's performance-unnecessary-copy-initialization over LLVM.
No functionality change intended. llvm-svn: 272516
This commit is contained in:
parent
b17b9a8324
commit
4ca41fd09e
|
@ -998,7 +998,7 @@ void DwarfDebug::beginInstruction(const MachineInstr *MI) {
|
|||
|
||||
// Check if source location changes, but ignore DBG_VALUE locations.
|
||||
if (!MI->isDebugValue()) {
|
||||
DebugLoc DL = MI->getDebugLoc();
|
||||
const DebugLoc &DL = MI->getDebugLoc();
|
||||
if (DL != PrevInstLoc) {
|
||||
if (DL) {
|
||||
unsigned Flags = 0;
|
||||
|
|
|
@ -1139,7 +1139,7 @@ bool HexagonAsmParser::parseOperand(OperandVector &Operands) {
|
|||
static char const *RParen = ")";
|
||||
Operands.push_back(HexagonOperand::CreateToken(LParen, Begin));
|
||||
Operands.push_back(HexagonOperand::CreateReg(Register, Begin, End));
|
||||
AsmToken MaybeDotNew = Lexer.getTok();
|
||||
const AsmToken &MaybeDotNew = Lexer.getTok();
|
||||
if (MaybeDotNew.is(AsmToken::TokenKind::Identifier) &&
|
||||
MaybeDotNew.getString().equals_lower(".new"))
|
||||
splitIdentifier(Operands);
|
||||
|
@ -1155,7 +1155,7 @@ bool HexagonAsmParser::parseOperand(OperandVector &Operands) {
|
|||
Operands.insert(Operands.end () - 1,
|
||||
HexagonOperand::CreateToken(LParen, Begin));
|
||||
Operands.push_back(HexagonOperand::CreateReg(Register, Begin, End));
|
||||
AsmToken MaybeDotNew = Lexer.getTok();
|
||||
const AsmToken &MaybeDotNew = Lexer.getTok();
|
||||
if (MaybeDotNew.is(AsmToken::TokenKind::Identifier) &&
|
||||
MaybeDotNew.getString().equals_lower(".new"))
|
||||
splitIdentifier(Operands);
|
||||
|
|
|
@ -1302,7 +1302,7 @@ bool RedundantInstrElimination::processBlock(MachineBasicBlock &B,
|
|||
continue;
|
||||
|
||||
// If found, replace the instruction with a COPY.
|
||||
DebugLoc DL = MI->getDebugLoc();
|
||||
const DebugLoc &DL = MI->getDebugLoc();
|
||||
const TargetRegisterClass *FRC = HBS::getFinalVRegClass(RD, MRI);
|
||||
unsigned NewR = MRI.createVirtualRegister(FRC);
|
||||
BuildMI(B, At, DL, HII.get(TargetOpcode::COPY), NewR)
|
||||
|
|
|
@ -960,7 +960,7 @@ void HexagonEarlyIfConversion::eliminatePhis(MachineBasicBlock *B) {
|
|||
// MRI.replaceVregUsesWith does not allow to update the subregister,
|
||||
// so instead of doing the use-iteration here, create a copy into a
|
||||
// "non-subregistered" register.
|
||||
DebugLoc DL = PN->getDebugLoc();
|
||||
const DebugLoc &DL = PN->getDebugLoc();
|
||||
const TargetRegisterClass *RC = MRI->getRegClass(DefR);
|
||||
NewR = MRI->createVirtualRegister(RC);
|
||||
NonPHI = BuildMI(*B, NonPHI, DL, TII->get(TargetOpcode::COPY), NewR)
|
||||
|
|
|
@ -616,7 +616,7 @@ MachineInstr *HexagonExpandCondsets::genCondTfrFor(MachineOperand &SrcOp,
|
|||
bool PredSense, bool ReadUndef, bool ImpUse) {
|
||||
MachineInstr *MI = SrcOp.getParent();
|
||||
MachineBasicBlock &B = *At->getParent();
|
||||
DebugLoc DL = MI->getDebugLoc();
|
||||
const DebugLoc &DL = MI->getDebugLoc();
|
||||
|
||||
// Don't avoid identity copies here (i.e. if the source and the destination
|
||||
// are the same registers). It is actually better to generate them here,
|
||||
|
|
|
@ -2120,7 +2120,7 @@ void HexagonFrameLowering::optimizeSpillSlots(MachineFunction &MF,
|
|||
MachineBasicBlock::iterator StartIt = SI, NextIt;
|
||||
MachineInstr *CopyIn = nullptr;
|
||||
if (SrcRR.Reg != FoundR || SrcRR.Sub != 0) {
|
||||
DebugLoc DL = SI->getDebugLoc();
|
||||
const DebugLoc &DL = SI->getDebugLoc();
|
||||
CopyIn = BuildMI(B, StartIt, DL, HII.get(TargetOpcode::COPY), FoundR)
|
||||
.addOperand(SrcOp);
|
||||
}
|
||||
|
|
|
@ -196,7 +196,7 @@ void HexagonRegisterInfo::eliminateFrameIndex(MachineBasicBlock::iterator II,
|
|||
// register and use it with offset 0.
|
||||
auto &MRI = MF.getRegInfo();
|
||||
unsigned TmpR = MRI.createVirtualRegister(&Hexagon::IntRegsRegClass);
|
||||
DebugLoc DL = MI.getDebugLoc();
|
||||
const DebugLoc &DL = MI.getDebugLoc();
|
||||
BuildMI(MB, II, DL, HII.get(Hexagon::A2_addi), TmpR)
|
||||
.addReg(BP)
|
||||
.addImm(RealOffset);
|
||||
|
|
|
@ -1191,7 +1191,7 @@ MachineBasicBlock *MipsTargetLowering::emitSignExtendToI32InReg(
|
|||
MachineInstr *MI, MachineBasicBlock *BB, unsigned Size, unsigned DstReg,
|
||||
unsigned SrcReg) const {
|
||||
const TargetInstrInfo *TII = Subtarget.getInstrInfo();
|
||||
DebugLoc DL = MI->getDebugLoc();
|
||||
const DebugLoc &DL = MI->getDebugLoc();
|
||||
|
||||
if (Subtarget.hasMips32r2() && Size == 1) {
|
||||
BuildMI(BB, DL, TII->get(Mips::SEB), DstReg).addReg(SrcReg);
|
||||
|
|
|
@ -1830,7 +1830,7 @@ eliminateCallFramePseudoInstr(MachineFunction &MF, MachineBasicBlock &MBB,
|
|||
unsigned LISInstr = is64Bit ? PPC::LIS8 : PPC::LIS;
|
||||
unsigned ORIInstr = is64Bit ? PPC::ORI8 : PPC::ORI;
|
||||
MachineInstr *MI = I;
|
||||
DebugLoc dl = MI->getDebugLoc();
|
||||
const DebugLoc &dl = MI->getDebugLoc();
|
||||
|
||||
if (isInt<16>(CalleeAmt)) {
|
||||
BuildMI(MBB, I, dl, TII.get(ADDIInstr), StackReg)
|
||||
|
|
|
@ -786,7 +786,7 @@ static const TableEntry PopTable[] = {
|
|||
///
|
||||
void FPS::popStackAfter(MachineBasicBlock::iterator &I) {
|
||||
MachineInstr* MI = I;
|
||||
DebugLoc dl = MI->getDebugLoc();
|
||||
const DebugLoc &dl = MI->getDebugLoc();
|
||||
ASSERT_SORTED(PopTable);
|
||||
if (StackTop == 0)
|
||||
report_fatal_error("Cannot pop empty stack!");
|
||||
|
|
|
@ -2504,7 +2504,7 @@ void X86InstrInfo::reMaterialize(MachineBasicBlock &MBB,
|
|||
llvm_unreachable("Unexpected instruction!");
|
||||
}
|
||||
|
||||
DebugLoc DL = Orig->getDebugLoc();
|
||||
const DebugLoc &DL = Orig->getDebugLoc();
|
||||
BuildMI(MBB, I, DL, get(X86::MOV32ri)).addOperand(Orig->getOperand(0))
|
||||
.addImm(Value);
|
||||
} else {
|
||||
|
|
|
@ -1302,7 +1302,7 @@ updateInlinedAtInfo(const DebugLoc &DL, DILocation *InlinedAtNode,
|
|||
/// to encode location where these instructions are inlined.
|
||||
static void fixupLineNumbers(Function *Fn, Function::iterator FI,
|
||||
Instruction *TheCall) {
|
||||
DebugLoc TheCallDL = TheCall->getDebugLoc();
|
||||
const DebugLoc &TheCallDL = TheCall->getDebugLoc();
|
||||
if (!TheCallDL)
|
||||
return;
|
||||
|
||||
|
|
|
@ -250,7 +250,7 @@ static int AsLexInput(SourceMgr &SrcMgr, MCAsmInfo &MAI,
|
|||
|
||||
bool Error = false;
|
||||
while (Lexer.Lex().isNot(AsmToken::Eof)) {
|
||||
AsmToken Tok = Lexer.getTok();
|
||||
const AsmToken &Tok = Lexer.getTok();
|
||||
|
||||
switch (Tok.getKind()) {
|
||||
default:
|
||||
|
|
|
@ -576,7 +576,7 @@ void AsmWriterEmitter::EmitGetRegisterName(raw_ostream &O) {
|
|||
O << " switch(AltIdx) {\n"
|
||||
<< " default: llvm_unreachable(\"Invalid register alt name index!\");\n";
|
||||
for (const Record *R : AltNameIndices) {
|
||||
std::string AltName(R->getName());
|
||||
const std::string &AltName = R->getName();
|
||||
std::string Prefix = !Namespace.empty() ? Namespace + "::" : "";
|
||||
O << " case " << Prefix << AltName << ":\n"
|
||||
<< " assert(*(AsmStrs" << AltName << "+RegAsmOffset"
|
||||
|
|
|
@ -709,7 +709,7 @@ int DFAPacketizerEmitter::collectAllComboFuncs(
|
|||
Record *ComboFunc = FuncData->getValueAsDef("TheComboFunc");
|
||||
const std::vector<Record*> &FuncList =
|
||||
FuncData->getValueAsListOfDefs("FuncList");
|
||||
std::string ComboFuncName = ComboFunc->getName();
|
||||
const std::string &ComboFuncName = ComboFunc->getName();
|
||||
unsigned ComboBit = FUNameToBitsMap[ComboFuncName];
|
||||
unsigned ComboResources = ComboBit;
|
||||
DEBUG(dbgs() << " combo: " << ComboFuncName
|
||||
|
|
Loading…
Reference in New Issue