[all-commits] [llvm/llvm-project] c24890: [OpenMP][NFC] Use pass by const ref for Dependenci...

Shafik Yaghmour via All-commits all-commits at lists.llvm.org
Tue May 13 09:09:58 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c24890305378979a66ea6d35bee754c03e61178f
      https://github.com/llvm/llvm-project/commit/c24890305378979a66ea6d35bee754c03e61178f
  Author: Shafik Yaghmour <shafik.yaghmour at intel.com>
  Date:   2025-05-13 (Tue, 13 May 2025)

  Changed paths:
    M llvm/include/llvm/Frontend/OpenMP/OMPIRBuilder.h
    M llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
    M llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp

  Log Message:
  -----------
  [OpenMP][NFC] Use pass by const ref for Dependencies (#139592)

Static analysis flagged the passing of Dependencies to emitTargetCall as
a
place we could use std::move to avoid copying. A closer look indicated
we could
instead turn the parameter into a const & and not have a default value
since it
was only used in two lines in a test and changing those two locations
was easy.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list