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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to