[llvm] r249511 - [RS4GC] Remove an unnecessary assert & related variables

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 19:39:27 PDT 2015


Author: sanjoy
Date: Tue Oct  6 21:39:27 2015
New Revision: 249511

URL: http://llvm.org/viewvc/llvm-project?rev=249511&view=rev
Log:
[RS4GC] Remove an unnecessary assert & related variables

I don't think this assert adds much value, and removing it and related
variables avoids an "unused variable" warning in release builds.

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

Modified: llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp?rev=249511&r1=249510&r2=249511&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/RewriteStatepointsForGC.cpp Tue Oct  6 21:39:27 2015
@@ -1338,11 +1338,6 @@ makeStatepointExplicitImpl(const CallSit
   assert(isStatepoint(CS) &&
          "This method expects to be rewriting a statepoint");
 
-  BasicBlock *BB = CS.getInstruction()->getParent();
-  Function *F = BB->getParent();
-  Module *M = F->getParent();
-  assert(M && "must be set");
-
   // We're not changing the function signature of the statepoint since the gc
   // arguments go into the var args section.
   Function *GCStatepointDecl = CS.getCalledFunction();




More information about the llvm-commits mailing list