diff options
author | Matthias Braun <matze@braunis.de> | 2014-12-11 19:42:05 +0000 |
---|---|---|
committer | Matthias Braun <matze@braunis.de> | 2014-12-11 19:42:05 +0000 |
commit | a7c82a9f1da9466ca0b5481e6ccb852a0fbba99d (patch) | |
tree | 7d45e944e048025936e2b6aa4acab90e69fa5665 /llvm/lib/Target/MSP430/MSP430TargetMachine.cpp | |
parent | a4e932db1656772da874f69ae63ab24a01038b62 (diff) | |
download | bcm5719-llvm-a7c82a9f1da9466ca0b5481e6ccb852a0fbba99d.tar.gz bcm5719-llvm-a7c82a9f1da9466ca0b5481e6ccb852a0fbba99d.zip |
[CodeGen] Add print and verify pass after each MachineFunctionPass by default
Previously print+verify passes were added in a very unsystematic way, which is
annoying when debugging as you miss intermediate steps and allows bugs to stay
unnotice when no verification is performed.
To make this change practical I added the possibility to explicitely disable
verification. I used this option on all places where no verification was
performed previously (because alot of places actually don't pass the
MachineVerifier).
In the long term these problems should be fixed properly and verification
enabled after each pass. I'll enable some more verification in subsequent
commits.
llvm-svn: 224042
Diffstat (limited to 'llvm/lib/Target/MSP430/MSP430TargetMachine.cpp')
-rw-r--r-- | llvm/lib/Target/MSP430/MSP430TargetMachine.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/llvm/lib/Target/MSP430/MSP430TargetMachine.cpp b/llvm/lib/Target/MSP430/MSP430TargetMachine.cpp index 8cee01677bf..6640b52f94a 100644 --- a/llvm/lib/Target/MSP430/MSP430TargetMachine.cpp +++ b/llvm/lib/Target/MSP430/MSP430TargetMachine.cpp @@ -50,7 +50,7 @@ public: } bool addInstSelector() override; - bool addPreEmitPass() override; + void addPreEmitPass() override; }; } // namespace @@ -64,8 +64,7 @@ bool MSP430PassConfig::addInstSelector() { return false; } -bool MSP430PassConfig::addPreEmitPass() { +void MSP430PassConfig::addPreEmitPass() { // Must run branch selection immediately preceding the asm printer. - addPass(createMSP430BranchSelectionPass()); - return false; + addPass(createMSP430BranchSelectionPass(), false); } |