[PATCH] D81179: [flang] avoid GCC < 8 compiler failure after D80794

[Valentin Clement via Phabricator via llvm-commits]

clementval added a comment.

In D81179#2074261 <https://reviews.llvm.org/D81179#2074261>, @DavidTruby wrote:

> This looks good to me.
>
> As an aside, I think this perfectly demonstrates why we should not force on Werror or have it on by default. We don't have public build bots for even the compilers we claim to support and seemingly innocent changes can cause spurious build failures for other users simply because we are forcing Werror on. And worse, at the moment we are forcing Werror on with no way to turn it off meaning these spurious errors can't even be avoided. This comment isn't intended in any way to block this patch; I just wanted to note it here.


Fully agree with your point on Werror. Thanks for the review.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81179/new/

https://reviews.llvm.org/D81179