https://github.com/HerrCai0907 approved this pull request.
LGTM with nits
1. follow LLVM coding guideline to remove `{}` for only one lines if / for / ...
2. personal I prefer to use different function name instead of `emitDiag` for
everythings.
https://github.com/llvm/llvm-project/pull/126434
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
