mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-19 11:02:59 +02:00
Correct misleading formatting of several ifs followed by two statements without braces.
While the original code would work with or without braces, it makes sense to set HaveSemi to true only if (!HaveSemi), otherwise it's already true, so I put the assignment inside the if block. This addresses PR25998. llvm-svn: 256688
This commit is contained in:
parent
a26d6b9c52
commit
5b0c84e633
@ -1738,7 +1738,10 @@ void MachineInstr::print(raw_ostream &OS, ModuleSlotTracker &MST,
|
||||
bool HaveSemi = false;
|
||||
const unsigned PrintableFlags = FrameSetup | FrameDestroy;
|
||||
if (Flags & PrintableFlags) {
|
||||
if (!HaveSemi) OS << ";"; HaveSemi = true;
|
||||
if (!HaveSemi) {
|
||||
OS << ";";
|
||||
HaveSemi = true;
|
||||
}
|
||||
OS << " flags: ";
|
||||
|
||||
if (Flags & FrameSetup)
|
||||
@ -1749,7 +1752,10 @@ void MachineInstr::print(raw_ostream &OS, ModuleSlotTracker &MST,
|
||||
}
|
||||
|
||||
if (!memoperands_empty()) {
|
||||
if (!HaveSemi) OS << ";"; HaveSemi = true;
|
||||
if (!HaveSemi) {
|
||||
OS << ";";
|
||||
HaveSemi = true;
|
||||
}
|
||||
|
||||
OS << " mem:";
|
||||
for (mmo_iterator i = memoperands_begin(), e = memoperands_end();
|
||||
@ -1762,7 +1768,10 @@ void MachineInstr::print(raw_ostream &OS, ModuleSlotTracker &MST,
|
||||
|
||||
// Print the regclass of any virtual registers encountered.
|
||||
if (MRI && !VirtRegs.empty()) {
|
||||
if (!HaveSemi) OS << ";"; HaveSemi = true;
|
||||
if (!HaveSemi) {
|
||||
OS << ";";
|
||||
HaveSemi = true;
|
||||
}
|
||||
for (unsigned i = 0; i != VirtRegs.size(); ++i) {
|
||||
const TargetRegisterClass *RC = MRI->getRegClass(VirtRegs[i]);
|
||||
OS << " " << TRI->getRegClassName(RC)
|
||||
@ -1781,7 +1790,8 @@ void MachineInstr::print(raw_ostream &OS, ModuleSlotTracker &MST,
|
||||
|
||||
// Print debug location information.
|
||||
if (isDebugValue() && getOperand(e - 2).isMetadata()) {
|
||||
if (!HaveSemi) OS << ";";
|
||||
if (!HaveSemi)
|
||||
OS << ";";
|
||||
auto *DV = cast<DILocalVariable>(getOperand(e - 2).getMetadata());
|
||||
OS << " line no:" << DV->getLine();
|
||||
if (auto *InlinedAt = debugLoc->getInlinedAt()) {
|
||||
@ -1795,7 +1805,8 @@ void MachineInstr::print(raw_ostream &OS, ModuleSlotTracker &MST,
|
||||
if (isIndirectDebugValue())
|
||||
OS << " indirect";
|
||||
} else if (debugLoc && MF) {
|
||||
if (!HaveSemi) OS << ";";
|
||||
if (!HaveSemi)
|
||||
OS << ";";
|
||||
OS << " dbg:";
|
||||
debugLoc.print(OS);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user