This revision was automatically updated to reflect the committed changes. Closed by commit rL279959: [InstructionSelect] NumBlocks isn't defined in DEBUG build. (authored by hokein).
Changed prior to commit: https://reviews.llvm.org/D23985?vs=69540&id=69542#toc Repository: rL LLVM https://reviews.llvm.org/D23985 Files: llvm/trunk/lib/CodeGen/GlobalISel/InstructionSelect.cpp Index: llvm/trunk/lib/CodeGen/GlobalISel/InstructionSelect.cpp =================================================================== --- llvm/trunk/lib/CodeGen/GlobalISel/InstructionSelect.cpp +++ llvm/trunk/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)) {
Index: llvm/trunk/lib/CodeGen/GlobalISel/InstructionSelect.cpp =================================================================== --- llvm/trunk/lib/CodeGen/GlobalISel/InstructionSelect.cpp +++ llvm/trunk/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)) {
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits