[PATCH] D56972: [CMake][Fuchsia] Drop -DNDEBUG, re-enable modules

2019-01-20 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL351709: [CMake][Fuchsia] Drop -DNDEBUG, re-enable modules (authored by phosek, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D56972?vs=182692

[PATCH] D56972: [CMake][Fuchsia] Drop -DNDEBUG, re-enable modules

2019-01-20 Thread Roland McGrath via Phabricator via cfe-commits
mcgrathr accepted this revision. mcgrathr added a comment. This revision is now accepted and ready to land. lgtm Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56972/new/ https://reviews.llvm.org/D56972 ___ cfe-commit

[PATCH] D56972: [CMake][Fuchsia] Drop -DNDEBUG, re-enable modules

2019-01-19 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: mcgrathr, jakehehrlich, juliehockett. Herald added subscribers: cfe-commits, mgorny. -DNDEBUG is no longer needed now that we don't enable assertions, modules should improve build times for the second stage. Repository: rC Clang https://r