diff options
| author | Stephen Hines <srhines@google.com> | 2017-08-25 00:48:21 +0000 |
|---|---|---|
| committer | Stephen Hines <srhines@google.com> | 2017-08-25 00:48:21 +0000 |
| commit | cc14a386d876a6dbcd9bb33b8dc6f0ed52f04486 (patch) | |
| tree | b784f38567f9cfc2eb5dc578f274a559c8fdb13d /llvm/test/CodeGen/X86 | |
| parent | 8ac488b16185a70f1f2589a3d0447ddedb7f73e3 (diff) | |
| download | bcm5719-llvm-cc14a386d876a6dbcd9bb33b8dc6f0ed52f04486.tar.gz bcm5719-llvm-cc14a386d876a6dbcd9bb33b8dc6f0ed52f04486.zip | |
Fix two (three) more issues with unchecked Error.
Summary:
If assertions are disabled, but LLVM_ABI_BREAKING_CHANGES is enabled,
this will cause an issue with an unchecked Success. Switching to
consumeError() is the correct way to bypass the check. This patch also
includes disabling 2 tests that can't work without assertions enabled,
since llvm_unreachable() with NDEBUG won't crash.
Reviewers: llvm-commits, lhames
Reviewed By: lhames
Subscribers: lhames, pirama
Differential Revision: https://reviews.llvm.org/D36729
llvm-svn: 311739
Diffstat (limited to 'llvm/test/CodeGen/X86')
0 files changed, 0 insertions, 0 deletions
