zmodem wrote: > I intend to make the warning on-by-default. However, it currently triggers at > several places in the LLVM project, breaking the CI. Fixing them is > nontrivial and cluttered up the PR, so I made it DefaultIgnore in order to > keep things focused. Should the PR be accepted I plan to go through and clean > things up, then default-enable the warning.
This sounds like a good approach. Can you share a few of the examples where it triggers, to get a feel for what this looks like on the LLVM code? https://github.com/llvm/llvm-project/pull/117622 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
