krasimir added a comment.

General comment:

I think this is useful and could also be useful for other languages too (some 
folks have asked for this for some C++ code).
As this is a non-whitespace only change, it's a bit more risky than 
whitespace-only changes.

I think we need to establish a policy on such more riskier style options, at 
least have a convenient way for users to to opt-out of such and be extra 
careful including them by default in default styles.
This is tricky because some style guides explicitly recommend such changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73354



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

Reply via email to