JonasToth added a comment.

How does this patch change the behaviour for macros without location? They will 
be diagnosed at the beginning of the TU?

It is probably better to ignore these kind of macros, as there is no way around 
them (compile time configuration can only be done with macros?!) and therefore 
warnings for theses macros are false positives. Using `constexpr` constructs 
does not help either.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53817



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

Reply via email to