[PATCH] D78306: [flang] Use LLVM's flags
David Truby via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 4 12:22:04 PDT 2020
DavidTruby added a comment.
In D78306#2018241 <https://reviews.llvm.org/D78306#2018241>, @tskeith wrote:
> With this change is it possible to enable `-Werror` for flang (in an in-tree build) without enabling it for all of LLVM? Also, `-pedantic`?
Could you let us know why you think this is necessary when LLVM_ENABLE_WERROR and LLVM_ENABLE_PEDANTIC exist?
It's perfectly acceptable to stand up build bots that enable these two flags and enforce that those build bots always pass in master; I just really don't think we want these flags by default when new compiler releases after our own releases (and therefore completely out of our control) can cause build failures with these flags.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78306/new/
https://reviews.llvm.org/D78306
More information about the llvm-commits
mailing list