[PATCH] D141569: [clang-tidy] Implement CppCoreGuideline F.18

Chris Cotter via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 23 12:19:22 PST 2023


ccotter added a comment.

> Imaginate that such trivial type could be for example 200KB in size

This should be passed by const ref then correct (listed under "Expensive to move (e.g. big BigPOD[]" in the parameter passing guidelines <http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#fcall-parameter-passing>) ? Are there cases where a big pod type needs to be passed by rvalue ref, without std::move where const ref cannot be used instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141569



More information about the cfe-commits mailing list