[PATCH] D87291: [clang-format][regression][PR47461] ifdef causes catch to be seen as a function

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 17 04:00:01 PDT 2020


MyDeveloperDay added a comment.

In D87291#2278743 <https://reviews.llvm.org/D87291#2278743>, @curdeius wrote:

> LGTM.
> Don't we risk any other side effects?

There could be, but maybe we can address them as we see them.

The original reason for including this was so that the Linux kernel could use `try` as a variable, I'm trying not to break that but I'm not convinced that

`int try /* abc */ = 0;`

would consider try as a variable any more. my hope is they don't do that..


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87291/new/

https://reviews.llvm.org/D87291



More information about the cfe-commits mailing list