kuganv added a comment. In D141950#4141455 <https://reviews.llvm.org/D141950#4141455>, @aaron.ballman wrote:
> LGTM! You should probably put "NFC" into the patch title when landing the > changes though, so it's more clear as to why there's no test coverage. > > Do you think this warrants adding a release note to let users know about the > performance improvement? Thanks for the review. I have updated the title with NFC. This may not be significant enough to require update on release notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141950/new/ https://reviews.llvm.org/D141950 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
