diff --git a/llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXInstPrinter.cpp b/llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXInstPrinter.cpp index d7a3daa450f0f6..45561d5a112386 100644 --- a/llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXInstPrinter.cpp +++ b/llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXInstPrinter.cpp @@ -228,9 +228,9 @@ void NVPTXInstPrinter::printLdStCode(const MCInst *MI, int OpNum, const MCOperand &MO = MI->getOperand(OpNum); int Imm = (int) MO.getImm(); if (!strcmp(Modifier, "sem")) { - auto ordering = + auto Ordering = NVPTX::Ordering(static_cast(Imm)); - switch (ordering) { + switch (Ordering) { case NVPTX::Ordering::NotAtomic: break; case NVPTX::Ordering::Volatile: @@ -251,15 +251,15 @@ void NVPTXInstPrinter::printLdStCode(const MCInst *MI, int OpNum, default: SmallString<256> Msg; raw_svector_ostream OS(Msg); - OS << "NVPTX LdStCode Printer does not support \"" << ordering + OS << "NVPTX LdStCode Printer does not support \"" << Ordering << "\" sem modifier."; report_fatal_error(OS.str()); break; } } else if (!strcmp(Modifier, "sc")) { - auto ordering = + auto Ordering = NVPTX::Ordering(static_cast(Imm)); - switch (ordering) { + switch (Ordering) { // TODO: refactor fence insertion in ISelDagToDag instead of here // as part of implementing atomicrmw seq_cst. case NVPTX::Ordering::SequentiallyConsistent: diff --git a/llvm/lib/Target/NVPTX/NVPTX.h b/llvm/lib/Target/NVPTX/NVPTX.h index 602ab6e150e2aa..c9cce23788ca4a 100644 --- a/llvm/lib/Target/NVPTX/NVPTX.h +++ b/llvm/lib/Target/NVPTX/NVPTX.h @@ -122,32 +122,32 @@ enum class Ordering : OrderingUnderlyingType { LAST = RelaxedMMIO }; -template OStream &operator<<(OStream &os, Ordering order) { - switch (order) { +template OStream &operator<<(OStream &O, Ordering Order) { + switch (Order) { case Ordering::NotAtomic: - os << "NotAtomic"; - return os; + O << "NotAtomic"; + return O; case Ordering::Relaxed: - os << "Relaxed"; - return os; + O << "Relaxed"; + return O; case Ordering::Acquire: - os << "Acquire"; - return os; + O << "Acquire"; + return O; case Ordering::Release: - os << "Release"; - return os; + O << "Release"; + return O; // case Ordering::AcquireRelease: - // os << "AcquireRelease"; - // return os; + // O << "AcquireRelease"; + // return O; case Ordering::SequentiallyConsistent: - os << "SequentiallyConsistent"; - return os; + O << "SequentiallyConsistent"; + return O; case Ordering::Volatile: - os << "Volatile"; - return os; + O << "Volatile"; + return O; case Ordering::RelaxedMMIO: - os << "RelaxedMMIO"; - return os; + O << "RelaxedMMIO"; + return O; } report_fatal_error("unknown ordering"); } diff --git a/llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp b/llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp index e23783bfde22eb..5666d52698e0c4 100644 --- a/llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp +++ b/llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp @@ -715,12 +715,12 @@ static unsigned int getCodeAddrSpace(MemSDNode *N) { } struct OperationOrderings { - NVPTX::OrderingUnderlyingType instr_ordering; - NVPTX::OrderingUnderlyingType fence_ordering; + NVPTX::OrderingUnderlyingType InstrOrdering; + NVPTX::OrderingUnderlyingType FenceOrdering; OperationOrderings(NVPTX::Ordering o = NVPTX::Ordering::NotAtomic, NVPTX::Ordering f = NVPTX::Ordering::NotAtomic) - : instr_ordering(static_cast(o)), - fence_ordering(static_cast(f)) {} + : InstrOrdering(static_cast(o)), + FenceOrdering(static_cast(f)) {} }; static OperationOrderings @@ -892,11 +892,11 @@ getOperationOrderings(MemSDNode *N, const NVPTXSubtarget *Subtarget) { // // This sets the ordering of the fence to SequentiallyConsistent, and // sets the corresponding ordering for the instruction. - NVPTX::Ordering ord; + NVPTX::Ordering InstrOrder; if (N->readMem()) { - ord = NVPTX::Ordering::Acquire; + InstrOrder = NVPTX::Ordering::Acquire; } else if (N->writeMem()) { - ord = NVPTX::Ordering::Release; + InstrOrder = NVPTX::Ordering::Release; } else { SmallString<256> Msg; raw_svector_ostream OS(Msg); @@ -907,7 +907,7 @@ getOperationOrderings(MemSDNode *N, const NVPTXSubtarget *Subtarget) { report_fatal_error(OS.str()); } return AddrGenericOrGlobalOrShared - ? OperationOrderings(ord, + ? OperationOrderings(InstrOrder, NVPTX::Ordering::SequentiallyConsistent) : OperationOrderings(NVPTX::Ordering::NotAtomic); }