[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