Typz added a comment. In D50078#2055227 <https://reviews.llvm.org/D50078#2055227>, @srj wrote:
> > And I also think it should be on by default instead of modifying many > > .clang-format files. So IMHO if we add an option, it should be opt-out. > > I can live with it being opt-out. My big concern is that (as it currently > stands) our project gets different 'canonical' clang-format results depending > on whether people are using LLVM10 vs LLVM11 (and both are common and > supported by our project), which is a headache. I just want a way to get > consistent, predictable formatting regardless of the clang-format version. Adding an option would be an issue there, I think: since clang-format is quite strict regarding options, the option would need to exist in both LLVM10 and LLVM11 if you need to support such use-case... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50078/new/ https://reviews.llvm.org/D50078 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
