From 01f2ca5612127b4ff44e3eb4a63da24627341df8 Mon Sep 17 00:00:00 2001 From: James Y Knight Date: Mon, 28 Mar 2016 15:05:30 +0000 Subject: [PATCH] NFC: skip FenceInst up-front in AtomicExpandPass. llvm-svn: 264583 --- llvm/lib/CodeGen/AtomicExpandPass.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/llvm/lib/CodeGen/AtomicExpandPass.cpp b/llvm/lib/CodeGen/AtomicExpandPass.cpp index c7e7efd..40140e4 100644 --- a/llvm/lib/CodeGen/AtomicExpandPass.cpp +++ b/llvm/lib/CodeGen/AtomicExpandPass.cpp @@ -86,9 +86,10 @@ bool AtomicExpand::runOnFunction(Function &F) { // Changing control-flow while iterating through it is a bad idea, so gather a // list of all atomic instructions before we start. - for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I) { - if (I->isAtomic()) - AtomicInsts.push_back(&*I); + for (inst_iterator II = inst_begin(F), E = inst_end(F); II != E; ++II) { + Instruction *I = &*II; + if (I->isAtomic() && !isa(I)) + AtomicInsts.push_back(I); } bool MadeChange = false; @@ -97,8 +98,7 @@ bool AtomicExpand::runOnFunction(Function &F) { auto SI = dyn_cast(I); auto RMWI = dyn_cast(I); auto CASI = dyn_cast(I); - assert((LI || SI || RMWI || CASI || isa(I)) && - "Unknown atomic instruction"); + assert((LI || SI || RMWI || CASI) && "Unknown atomic instruction"); if (TLI->shouldInsertFencesForAtomic(I)) { auto FenceOrdering = Monotonic; -- 2.7.4