diff options
author | Reid Kleckner <rnk@google.com> | 2023-08-22 18:10:41 -0700 |
---|---|---|
committer | Reid Kleckner <rnk@google.com> | 2023-08-22 18:10:41 -0700 |
commit | 0d9919d362a7a70b2a7970861d897ecc47ec9e4d (patch) | |
tree | 06e297628d42e9a7562a98fba12dde2f4fd70ff3 /flang/lib/Frontend/CompilerInvocation.cpp | |
parent | d0ec03a384a5aa73a51735e897fecac0e0ad13b1 (diff) | |
download | llvm-0d9919d362a7a70b2a7970861d897ecc47ec9e4d.zip llvm-0d9919d362a7a70b2a7970861d897ecc47ec9e4d.tar.gz llvm-0d9919d362a7a70b2a7970861d897ecc47ec9e4d.tar.bz2 |
Revert "[Clang] CWG1473: do not err on the lack of space after operator"""
This reverts commit f2583f3acf596cc545c8c0e3cb28e712f4ebf21b.
There is a large body of non-conforming C-like code using format strings
like this:
#define PRIuS "zu"
void h(size_t foo, size_t bar) {
printf("foo is %"PRIuS", bar is %"PRIuS, foo, bar);
}
Rejecting this code would be very disruptive. We could decide to do
that, but it's sufficiently disruptive that I think it requires
gathering more community consensus with an RFC, and Aaron indicated [1]
it's OK to revert for now so continuous testing systems can see past
this issue while we decide what to do.
[1] https://reviews.llvm.org/D153156#4607717
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions