diff options
| author | Tom Stellard <thomas.stellard@amd.com> | 2014-09-25 18:59:22 +0000 |
|---|---|---|
| committer | Tom Stellard <thomas.stellard@amd.com> | 2014-09-25 18:59:22 +0000 |
| commit | 529efcf9d055bf78e678d4cbf6027f4737d4fd0d (patch) | |
| tree | 7ac52ca328d247e4f2b7084f8151881cb5ad5431 /llvm/lib/CodeGen/SelectionDAG | |
| parent | 7980fc856272f392bba56f0e09cec5475b8a5090 (diff) | |
| download | bcm5719-llvm-529efcf9d055bf78e678d4cbf6027f4737d4fd0d.tar.gz bcm5719-llvm-529efcf9d055bf78e678d4cbf6027f4737d4fd0d.zip | |
SelectionDAG: Remove #if NDEBUG from check for a post-isel hook
The InstrEmitter will skip the check of MI.hasPostISelHook()
before calling AdjustInstrPostInstrSelection() when NDEBUG
is not defined.
This was added in r140228, and I'm not sure if it is intentional or not,
but it is a likely source for bugs, because it means with
Release+Asserts builds you can forget to set the hasPostISelHook
flag on TableGen definitions and AdjustInstrPostInstrSelection() will
still be called.
llvm-svn: 218458
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp b/llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp index 6661aca6feb..4d6de12161c 100644 --- a/llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp @@ -865,9 +865,7 @@ EmitMachineNode(SDNode *Node, bool IsClone, bool IsCloned, MIB->setPhysRegsDeadExcept(UsedRegs, *TRI); // Run post-isel target hook to adjust this instruction if needed. -#ifdef NDEBUG if (II.hasPostISelHook()) -#endif TLI->AdjustInstrPostInstrSelection(MIB, Node); } |
