[all-commits] [llvm/llvm-project] 4f763b: [llvm][NFC] Hide the tensorflow dependency from he...

[Mircea Trofin via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 4f763b2172c591ab253c8489fcd53af0c544d5cb
      https://github.com/llvm/llvm-project/commit/4f763b2172c591ab253c8489fcd53af0c544d5cb
  Author: Mircea Trofin <mtrofin at google.com>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/Utils/TFUtils.h
    M llvm/lib/Analysis/InlineSizeEstimatorAnalysis.cpp
    M llvm/lib/Analysis/TFUtils.cpp
    M llvm/unittests/Analysis/TFUtilsTest.cpp

  Log Message:
  -----------
  [llvm][NFC] Hide the tensorflow dependency from headers.

Summary:
This change avoids exposing tensorflow types when including TFUtils.h.
They are just an implementation detail, and don't need to be used
directly when implementing an analysis requiring ML model evaluation.

The TFUtils APIs, while generically typed, are still not exposed unless
the tensorflow C library is present, as they currently have no use
otherwise.

Reviewers: mehdi_amini, davidxl

Subscribers: hiraditya, llvm-commits

Tags: #llvm

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