diff options
| author | Daniel Sanders <daniel_l_sanders@apple.com> | 2017-01-06 14:29:34 +0000 |
|---|---|---|
| committer | Daniel Sanders <daniel_l_sanders@apple.com> | 2017-01-06 14:29:34 +0000 |
| commit | 12360efa7623b1f279ce02c7f36895b1a65cb6f5 (patch) | |
| tree | 7d918a7a3ad41cba8f1b562fefc29308e8134178 /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp | |
| parent | c523c38fcc829a14ea77766f7d177266206fb276 (diff) | |
| download | llvm-12360efa7623b1f279ce02c7f36895b1a65cb6f5.zip llvm-12360efa7623b1f279ce02c7f36895b1a65cb6f5.tar.gz llvm-12360efa7623b1f279ce02c7f36895b1a65cb6f5.tar.bz2 | |
[globalisel] Stop requiring -debug/-debug-only=registerbankinfo for assertions.
Summary:
I've noticed that these assertions don't trigger when the condition is false.
The problem is that the DEBUG(x) macro only executes x when the pass is
emitting debug output via the -debug and -debug-only=registerbankinfo command
line arguments.
Debug builds should always execute the assertions so use '#ifndef NDEBUG' instead.
Also removed an assertion that is only true the first time it's tested. <Target>RegisterBankInfo's constructor will re-use register banks causing them to be valid on subsequent tests. That
assertion will fail on the first test too in the near future.
Reviewers: t.p.northover, ab, rovka, qcolombet
Subscribers: dberris, llvm-commits, kristof.beyls
Differential Revision: https://reviews.llvm.org/D28358
llvm-svn: 291235
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions