[all-commits] [llvm/llvm-project] 8dc3fe: [NFC][MLGO] Use std::move when moving protobufs

Mircea Trofin via All-commits all-commits at lists.llvm.org
Fri Aug 20 13:41:00 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8dc3fe0cd1e1353e39be032a94bb163f7d4e0e14
      https://github.com/llvm/llvm-project/commit/8dc3fe0cd1e1353e39be032a94bb163f7d4e0e14
  Author: Mircea Trofin <mtrofin at google.com>
  Date:   2021-08-20 (Fri, 20 Aug 2021)

  Changed paths:
    M llvm/lib/Analysis/TFUtils.cpp

  Log Message:
  -----------
  [NFC][MLGO] Use std::move when moving protobufs

Because of an odd linking problem, we need to temporarily support
building with TF C API 1.15 + tensorflow 2.50 pip package in
'development' mode scenarios. Protobuf Message 'Swap' is partially
implemented in the header (2.50) and relies on a symbol not found in TF
C API 1.15. std::move avoids that, at no semantic cost.




More information about the All-commits mailing list