[Lldb-commits] [PATCH] D65939: [lldb][CMake] Add LLDB_ENABLE_WERROR option
Stefan Gränitz via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 8 06:34:37 PDT 2019
sgraenitz abandoned this revision.
sgraenitz added a comment.
In D65939#1620780 <https://reviews.llvm.org/D65939#1620780>, @labath wrote:
> So, why doesn't LLVM_ENABLE_WERROR suffice? It looks like that ought to work even in standalone builds...
LLVM_ENABLE_WERROR affects the entire build tree. If another subproject causes a warning the build fails. In a way, you are right that this is sufficient, because the no-warnings policy seems to work well upstream.
The issue I was trying to solve is, that we can't use the setting with some external subprojects, because they are not strictly warning free. But then it may be better to have downstream. I agree.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits