diff options
| author | Matt Beaumont-Gay <matthewbg@google.com> | 2012-10-24 01:14:28 +0000 |
|---|---|---|
| committer | Matt Beaumont-Gay <matthewbg@google.com> | 2012-10-24 01:14:28 +0000 |
| commit | 6d919fb67bf6aa3db09608fb2948b558977c6929 (patch) | |
| tree | e955eb1aa915754e9c25d3878c6c5e42c7e55b40 /test/Sema/unused-expr.c | |
| parent | fa60be028105ded3c05d9fcc6cb0974500a5a011 (diff) | |
Address feedback from Eli Friedman on r166522.
In particular, we do want to warn on some unused cast subexpressions within
macros.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@166534 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Sema/unused-expr.c')
| -rw-r--r-- | test/Sema/unused-expr.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/test/Sema/unused-expr.c b/test/Sema/unused-expr.c index 6677e48300..aa81febdbb 100644 --- a/test/Sema/unused-expr.c +++ b/test/Sema/unused-expr.c @@ -123,9 +123,13 @@ void f(int i, ...) { // PR8371 int fn5() __attribute__ ((__const)); -// OpenSSL has some macros like this. -#define M(a, b) (long)foo((a), (b)) +// OpenSSL has some macros like this; we shouldn't warn on the cast. +#define M1(a, b) (long)foo((a), (b)) +// But, we should still warn on other subexpressions of casts in macros. +#define M2 (long)0; void t11(int i, int j) { - M(i, j); // no warning + M1(i, j); // no warning + M2; // expected-warning {{expression result unused}} } -#undef M +#undef M1 +#undef M2 |