alexfh added a comment.

In http://reviews.llvm.org/D14204#278833, @angelgarcia wrote:

> Use Lexer::makeFileCharRange (and a few changes due to clang-format).
>
> It seems to work as well.


Can you add a test where a void argument is removed in a macro (which would be 
the whole point of using the makeFileCharRange instead of just bailing out on 
all macros)?

E.g. does this work in a simple case like this one?

  #define M(x) x
  M(void f(void) {})
  // CHECK-FIXES: M(void f() {})


http://reviews.llvm.org/D14204



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

Reply via email to