diff options
| author | Eli Friedman <efriedma@quicinc.com> | 2021-06-21 16:34:02 -0700 |
|---|---|---|
| committer | Eli Friedman <efriedma@quicinc.com> | 2021-06-21 16:49:27 -0700 |
| commit | 74909e4b6e9bc0da6c197cf6c4419991a8dc335f (patch) | |
| tree | 64c3f062509a3ad2866167960fedb47d40e47a23 /llvm/lib/CodeGen/MachineStableHash.cpp | |
| parent | ac15a128d8756074b150bb5cb11bfd93a1a5c30c (diff) | |
| download | llvm-74909e4b6e9bc0da6c197cf6c4419991a8dc335f.zip llvm-74909e4b6e9bc0da6c197cf6c4419991a8dc335f.tar.gz llvm-74909e4b6e9bc0da6c197cf6c4419991a8dc335f.tar.bz2 | |
Rename MachineMemOperand::getOrdering -> getSuccessOrdering.
Since this method can apply to cmpxchg operations, make sure it's clear
what value we're actually retrieving. This will help ensure we don't
accidentally ignore the failure ordering of cmpxchg in the future.
We could potentially introduce a getOrdering() method on AtomicSDNode
that asserts the operation isn't cmpxchg, but not sure that's
worthwhile.
Differential Revision: https://reviews.llvm.org/D103338
Diffstat (limited to 'llvm/lib/CodeGen/MachineStableHash.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/MachineStableHash.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/MachineStableHash.cpp b/llvm/lib/CodeGen/MachineStableHash.cpp index fb14f0a..0803c2b 100644 --- a/llvm/lib/CodeGen/MachineStableHash.cpp +++ b/llvm/lib/CodeGen/MachineStableHash.cpp @@ -182,7 +182,7 @@ stable_hash llvm::stableHashValue(const MachineInstr &MI, bool HashVRegs, HashComponents.push_back(static_cast<unsigned>(Op->getSize())); HashComponents.push_back(static_cast<unsigned>(Op->getFlags())); HashComponents.push_back(static_cast<unsigned>(Op->getOffset())); - HashComponents.push_back(static_cast<unsigned>(Op->getOrdering())); + HashComponents.push_back(static_cast<unsigned>(Op->getSuccessOrdering())); HashComponents.push_back(static_cast<unsigned>(Op->getAddrSpace())); HashComponents.push_back(static_cast<unsigned>(Op->getSyncScopeID())); HashComponents.push_back(static_cast<unsigned>(Op->getBaseAlign().value())); |