[all-commits] [llvm/llvm-project] 59cb47: [NFC][clang] Fix Coverity static analyzer tool con...

[smanna12 via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 59cb47015a184862f3709a7ab084ccd086816176
      https://github.com/llvm/llvm-project/commit/59cb47015a184862f3709a7ab084ccd086816176
  Author: Manna, Soumi <soumi.manna at intel.com>
  Date:   2023-04-05 (Wed, 05 Apr 2023)

  Changed paths:
    M clang/lib/AST/ComputeDependence.cpp
    M clang/lib/Frontend/FrontendActions.cpp

  Log Message:
  -----------
  [NFC][clang] Fix Coverity static analyzer tool concerns about auto_causes_copy

    Reported by Coverity:

    AUTO_CAUSES_COPY
    Unnecessary object copies can affect performance

    Inside FrontendActions.cpp file,
    In clang::DumpModuleInfoAction::ExecuteAction(): Using the auto keyword without an & causes the copy of an object of type pair.

    Inside ComputeDependence.cpp file,
    In clang::computeDependence(clang::OverloadExpr *, bool, bool, bool): Using the auto keyword without an & causes the copy of an object of type TemplateArgumentLoc.

    Reviewed By: erichkeane, aaron.ballman

    Differential Revision: https://reviews.llvm.org/D147574