[PATCH] D112648: [clang-tidy] Improve fix-its for `modernize-pass-by-value` check

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 15 13:28:33 PST 2021


aaron.ballman added reviewers: aaron.ballman, whisperity, hokein.
aaron.ballman added a comment.
Herald added a subscriber: rnkovacs.

In D112648#3132157 <https://reviews.llvm.org/D112648#3132157>, @avogelsgesang wrote:

> @ymandel,  @whisperity, @aaron.ballman could one of you review this/point me in the direction of a good reviewer for this change?
> (Sorry for the spam - I am new to the LLVM project, and I guess I still have to learn how to navigate Phabricator/find the correct reviewers for my changes)

No worries; this is not spam, it was super handy for raising awareness!

I've added a few more reviewers who can hopefully help. I'll try to take a look once I get the chance, but I'm pretty swamped currently with C standards meetings this week. Also, the LLVM dev conference is this week, so reviewers may be hard to come by. Thank you for your patience though!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112648/new/

https://reviews.llvm.org/D112648



More information about the cfe-commits mailing list