[PATCH] D131057: [Sema] -Wformat: support C23 printf %b %B
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 3 01:17:42 PDT 2022
MaskRay created this revision.
MaskRay added reviewers: aaron.ballman, dim, enh, erik.pilkington.
Herald added a subscriber: StephenFan.
Herald added a project: All.
MaskRay requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Close #56885. %b %B are from WG14 N2630 and supported by glibc from 2.35 onwards and latest Android bionic.
Like GCC, we don't test library support.
GCC 12 -Wformat -pedantic emits a warning:
> warning: ISO C17 does not support the ‘%b’ gnu_printf format [-Wformat=]
The behavior is not ported (and it's unclear to me how to implement it).
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D131057
Files:
clang/include/clang/AST/FormatString.h
clang/lib/AST/FormatString.cpp
clang/lib/AST/PrintfFormatString.cpp
clang/test/Sema/format-strings-fixit.c
clang/test/Sema/format-strings.c
clang/test/SemaObjC/format-strings-objc.m
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131057.449575.patch
Type: text/x-patch
Size: 8200 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220803/b756f681/attachment.bin>
More information about the cfe-commits
mailing list