Author: resistor
Date: Mon Jul 23 17:05:54 2007
New Revision: 40448

URL: http://llvm.org/viewvc/llvm-project?rev=40448&view=rev
Log:
Don't delete volatile loads.  Doing so is not safe.

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

Modified: llvm/trunk/lib/Transforms/Scalar/FastDLE.cpp
URL: 
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/FastDLE.cpp?rev=40448&r1=40447&r2=40448&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/FastDLE.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/FastDLE.cpp Mon Jul 23 17:05:54 2007
@@ -69,6 +69,12 @@
   for (BasicBlock::iterator BBI = BB.begin(), BBE = BB.end(); BBI != BBE; 
++BBI) {
     // If we find a store or a free...
     if (LoadInst* L = dyn_cast<LoadInst>(BBI)) {
+      // We can't delete volatile loads
+      if (L->isVolatile()) {
+        lastLoad[L->getPointerOperand()] = L;
+        continue;
+      }
+      
       Value* pointer = L->getPointerOperand();
       LoadInst*& last = lastLoad[pointer];
       


_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to