Skip to content

Commit

Permalink
[lld][COFF][LTO] Don't disable verifier in assert builds
Browse files Browse the repository at this point in the history
We should catch more issues this way.

This previously the behavior and was dropped in cde5e5b.

Reviewed By: hans

Differential Revision: https://reviews.llvm.org/D134839
  • Loading branch information
aeubanks committed Sep 29, 2022
1 parent 6c8d8d1 commit 1079f1c
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions lld/COFF/LTO.cpp
Expand Up @@ -76,7 +76,11 @@ static lto::Config createConfig() {
c.RelocModel = Reloc::Static;
else
c.RelocModel = Reloc::PIC_;
#ifndef NDEBUG
c.DisableVerify = false;
#else
c.DisableVerify = true;
#endif
c.DiagHandler = diagnosticHandler;
c.OptLevel = config->ltoo;
c.CPU = getCPUStr();
Expand Down

0 comments on commit 1079f1c

Please sign in to comment.