[PATCH] D131727: Enable -Wctad-maybe-unsupported in LLVM build

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 10:50:56 PDT 2022


dblaikie added a comment.

In D131727#3726568 <https://reviews.llvm.org/D131727#3726568>, @kiranchandramohan wrote:

> @dblaikie Since this change causes Flang buildbots to fail and would require more than a few changes in Flang, would it be OK to revert and have the changes in Flang be reviewed before resubmitting?

I'm particularly interested in avoiding backsliding in LLVM if we were to disable the warning entirely & seeing as Flang has broad use of the feature already I've submitted 357e99aff035e981f7d7fc566ffa785925594259 <https://reviews.llvm.org/rG357e99aff035e981f7d7fc566ffa785925594259> to disable the warning in flang & two follow-up commits to revert my incomplete attempts to address the issue in the Flang codebase - if you/the Flang developers decide the warning is worthwhile, that can now be done independently of ensuring LLVM doesn't regress this constraint. I'm happy to help/discuss/provide examples/etc if that's of interest.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131727



More information about the llvm-commits mailing list