[PATCH] Add AtomicExpandPass::bracketInstWithFences, and use it whenever getInsertFencesForAtomic would trigger in SelectionDAGBuilder

Reid Kleckner rnk at google.com
Tue Sep 23 11:24:44 PDT 2014


================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:11052
@@ -11052,2 +11051,3 @@
   }
+  llvm_unreachable("Unknown fence ordering in emitLeadingFence");
 }
----------------
jfb wrote:
> Fold this into the `default` case for the `switch`.
We can't do this because MSVC will warn about falling off the end of the function.

http://reviews.llvm.org/D5179






More information about the llvm-commits mailing list