[PATCH] D23985: [InstructionSelect] NumBlocks isn't defined in DEBUG build.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 29 01:43:49 PDT 2016


hokein created this revision.
hokein added a reviewer: bkramer.
hokein added a subscriber: cfe-commits.

A follow-up fixing on http://llvm.org/viewvc/llvm-project?view=revision&revision=279905.

https://reviews.llvm.org/D23985

Files:
  lib/CodeGen/GlobalISel/InstructionSelect.cpp

Index: lib/CodeGen/GlobalISel/InstructionSelect.cpp
===================================================================
--- lib/CodeGen/GlobalISel/InstructionSelect.cpp
+++ lib/CodeGen/GlobalISel/InstructionSelect.cpp
@@ -81,10 +81,10 @@
         if (isPreISelGenericOpcode(MI.getOpcode()) && !MLI->isLegal(MI))
           reportSelectionError(MI, "Instruction is not legal");
 
+#endif
   // FIXME: We could introduce new blocks and will need to fix the outer loop.
   // Until then, keep track of the number of blocks to assert that we don't.
   const size_t NumBlocks = MF.size();
-#endif
 
   bool Failed = false;
   for (MachineBasicBlock *MBB : post_order(&MF)) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23985.69540.patch
Type: text/x-patch
Size: 670 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160829/60e6c658/attachment-0001.bin>


More information about the cfe-commits mailing list