[llvm-commits] [llvm] r110435 - /llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp

Nick Lewycky nicholas at mxc.ca
Fri Aug 6 00:43:46 PDT 2010


Author: nicholas
Date: Fri Aug  6 02:43:46 2010
New Revision: 110435

URL: http://llvm.org/viewvc/llvm-project?rev=110435&view=rev
Log:
Fix uninitialized variable warning.
Also move 'default' case next to a real case to help compiler optimize in
non-Debug builds.
No functionality change.

Modified:
    llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp?rev=110435&r1=110434&r2=110435&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LowerAtomic.cpp Fri Aug  6 02:43:46 2010
@@ -52,8 +52,9 @@
     Value *Delta = CI->getArgOperand(1);
 
     LoadInst *Orig = Builder.CreateLoad(Ptr);
-    Value *Res;
+    Value *Res = NULL;
     switch (IID) {
+      default: assert(0 && "Unrecognized atomic modify operation");
       case Intrinsic::atomic_load_add:
         Res = Builder.CreateAdd(Orig, Delta);
         break;
@@ -92,7 +93,6 @@
                                    Orig,
                                    Delta);
         break;
-      default: assert(0 && "Unrecognized atomic modify operation");
     }
     Builder.CreateStore(Res, Ptr);
 





More information about the llvm-commits mailing list