================
Comment at: lib/Sema/SemaStmt.cpp:169
@@ +168,3 @@
+  if (SourceMgr.isMacroBodyExpansion(ExprLoc) &&
+      !(isa<CallExpr>(E->IgnoreParens()) &&
+        E->getLocStart().isMacroID() &&
----------------
The ad-hoc nature of this check gives me pause. I'd be more comfortable saying 
that __attribute__((warn_unused_result)) always gives a warning, even in a 
macro body.


http://llvm-reviews.chandlerc.com/D380
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to