diff options
author | Tim Northover <tnorthover@apple.com> | 2013-07-01 18:37:33 +0000 |
---|---|---|
committer | Tim Northover <tnorthover@apple.com> | 2013-07-01 18:37:33 +0000 |
commit | 7f3d9e1f36fdfe6de61a3fe58bb6d1fed61adca0 (patch) | |
tree | a99177bbec9e26124dbd5b64eb0e93cb998978fa /llvm/lib/Target/ARM/ARMISelLowering.cpp | |
parent | 08016a657ceb894928a909c39cc5ceee85a0e20b (diff) | |
download | bcm5719-llvm-7f3d9e1f36fdfe6de61a3fe58bb6d1fed61adca0.tar.gz bcm5719-llvm-7f3d9e1f36fdfe6de61a3fe58bb6d1fed61adca0.zip |
Revert r185339 (ARM: relax the atomic release barrier to "dmb ishst")
Turns out I'd misread the architecture reference manual and thought
that was a load/store-store barrier, when it's not.
Thanks for pointing it out Eli!
llvm-svn: 185356
Diffstat (limited to 'llvm/lib/Target/ARM/ARMISelLowering.cpp')
-rw-r--r-- | llvm/lib/Target/ARM/ARMISelLowering.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp index 370962dfa5d..ff8571ba033 100644 --- a/llvm/lib/Target/ARM/ARMISelLowering.cpp +++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp @@ -2557,12 +2557,8 @@ static SDValue LowerATOMIC_FENCE(SDValue Op, SelectionDAG &DAG, DAG.getConstant(0, MVT::i32)); } - ConstantSDNode *OrdN = cast<ConstantSDNode>(Op.getOperand(1)); - AtomicOrdering Ord = static_cast<AtomicOrdering>(OrdN->getZExtValue()); - unsigned Domain = Ord == Release ? ARM_MB::ISHST : ARM_MB::ISH; - return DAG.getNode(ARMISD::MEMBARRIER, dl, MVT::Other, Op.getOperand(0), - DAG.getConstant(Domain, MVT::i32)); + DAG.getConstant(ARM_MB::ISH, MVT::i32)); } static SDValue LowerPREFETCH(SDValue Op, SelectionDAG &DAG, |