On Fri, Feb 01, 2019 at 10:05:02AM +0100, Richard Biener wrote: > We definitely shouldn't willingly add new instances. Why do we care > about people getting false positives when they willingly disable > -fprintf-return-value?
The -fprintf-return-value switch is about code transformations, perhaps somebody wants warnings but doesn't want the return value to be ignored or assumed to be such and such (e.g. because his C library doesn't do exactly what GCC is expecting it to do). Anyway, not that important to me, if you prefer adding "&& flag_printf_return_value" to Aldy's changes, let's go ahead with that. Jakub