vvuksanovic wrote:

> As far as I'm concerned, `-Wmissing-format-attribute` is the subset of 
> `-Wformat-nonliteral` that we can fix it by adding a format attribute. This 
> change is valuable and I want it to land, but I think that we need to close 
> the gaps with `-Wformat-nonliteral` to make sure we're consistent. This 
> mostly means ensuring you don't get both a `-Wmissing-format-attribute` and a 
> `-Wformat-nonliteral` diagnostic for the same code. And because of that 
> relationship, I think we should also make `-Wmissing-format-attribute` a 
> subgroup of `-Wformat-nonliteral`, as it will subsume diagnostics that 
> -Wformat-nonliteral previously emitted.

Let me make sure I understand. The idea is to emit `-Wmissing-format-attribute` 
instead of `-Wformat-nonliteral` when possible, otherwise `-Wformat-nonliteral` 
would trigger. If so, I agree with that.


https://github.com/llvm/llvm-project/pull/166738
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to