MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.

I'm not a fan of this approach of adding a "C" language, mainly because of the 
`.h` problem so ultimately it doesn't solve your problem.

I think this is overkill for what is basically the subtle handling of 
"public/private/protected/try/delete/new" being used as variables (which 
frankly is at least in my view not a bright idea in the first place!)

I think ultimately we can try to catch those corner cases but IMHO it doesn't 
need the addition of a whole new language for that, (we are not at that much on 
an impasse)

This review contains .arclint that needs removing, this review contains no tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117416

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

Reply via email to