}
case CXXFunctionalCastExprClass:
case CStyleCastExprClass: {
- // Ignore casts within macro expansions.
- if (getExprLoc().isMacroID())
- return false;
-
// Ignore an explicit cast to void unless the operand is a non-trivial
// volatile lvalue.
const CastExpr *CE = cast<CastExpr>(this);
return false;
}
+ // Ignore casts within macro expansions.
+ if (getExprLoc().isMacroID())
+ return CE->getSubExpr()->isUnusedResultAWarning(WarnE, Loc, R1, R2, Ctx);
+
// If this is a cast to a constructor conversion, check the operand.
// Otherwise, the result of the cast is unused.
if (CE->getCastKind() == CK_ConstructorConversion)
// 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