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

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 20 02:10:42 PST 2017


ilya-biryukov added a comment.

In https://reviews.llvm.org/D40185#928845, @rnk wrote:

> Why does passing the rparen location for the if condition fix the warning for IF_ELSE? I assumed that would refer to the else clause semicolon.


Using spelling locations fixes the `else` case, but breaks some of the  `if ` cases involving macros (catched by `warn-empty-body.cpp` test).


Repository:
  rL LLVM

https://reviews.llvm.org/D40185





More information about the cfe-commits mailing list