MaskRay added a comment. In D99308#2649636 <https://reviews.llvm.org/D99308#2649636>, @raj.khem wrote:
> once you fix the clang-format issue reported this looks good to me. I'll ignore it... The first block does not directly touch the formatted part and the second block would need a larger formatting... Such formatting changes should be done separately, if done at all, not mixed in this functionality patch. Personally I deem the formatting less useful, so I'll ignore it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99308/new/ https://reviews.llvm.org/D99308 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
