aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

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?


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
  • [PATCH] D141950: Us... Kugan Vivekanandarajah via Phabricator via cfe-commits
    • [PATCH] D14195... Kugan Vivekanandarajah via Phabricator via cfe-commits
    • [PATCH] D14195... Kugan Vivekanandarajah via Phabricator via cfe-commits
    • [PATCH] D14195... Aaron Ballman via Phabricator via cfe-commits

Reply via email to