[PATCH] D40185: Loosen -Wempty-body warning.

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 17 12:53:39 PST 2017


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

I'll go ahead and land this as requested to unblock things. Thanks for the patch, sorry for the trouble!



================
Comment at: lib/Sema/SemaChecking.cpp:11818
   bool StmtLineInvalid;
-  unsigned StmtLine = SourceMgr.getPresumedLineNumber(StmtLoc,
+  unsigned StmtLine = SourceMgr.getSpellingLineNumber(StmtLoc,
                                                       &StmtLineInvalid);
----------------
I see, this fixes the else macro case. 


https://reviews.llvm.org/D40185





More information about the cfe-commits mailing list