[PATCH] D44765: PR36643 Make clang-format support more generic TMarcos

Teodor Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 22 10:21:38 PDT 2018


obfuscated added a comment.

In https://reviews.llvm.org/D44765#1045373, @alexfh wrote:

> We can't just treat `anything("....")` like the _T macro. There should be a whitelist configurable with an option. By default only _T should be handled.


What cases could break with this version of the patch?
Or you think it is just too dangerous to have this in such generic form?
I'm fine adding an option if this is deemed acceptable.


Repository:
  rC Clang

https://reviews.llvm.org/D44765





More information about the cfe-commits mailing list