[PATCH] D117611: [Sema] Warn about printf %n on Android and Fuchsia

2022-02-23 Thread Elliott Hughes via Phabricator via cfe-commits
enh added a comment. In D117611#3339137 , @glandium wrote: > This doesn't leave much room to use `__attribute__((format(printf)))` on > custom printf implementations that do support `%n` on Android does it? it would be pretty hard to get into that situa

[PATCH] D117611: [Sema] Warn about printf %n on Android and Fuchsia

2022-02-23 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D117611#3339137 , @glandium wrote: > This doesn't leave much room to use `__attribute__((format(printf)))` on > custom printf implementations that do support `%n` on Android does it? +1 to this point; whether this is or

[PATCH] D117611: [Sema] Warn about printf %n on Android and Fuchsia

2022-02-22 Thread Mike Hommey via Phabricator via cfe-commits
glandium added a comment. This doesn't leave much room to use `__attribute__((format(printf)))` on custom printf implementations that do support `%n` on Android does it? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117611/new/ https://reviews.llv

[PATCH] D117611: [Sema] Warn about printf %n on Android and Fuchsia

2022-01-21 Thread Alex Brachet via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGcd4e600f5f5c: [Sema] Warn about printf %n on Android and Fuchsia (authored by abrachet). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo CHANG