[PATCH] D86549: [MLInliner] Simplify TFUTILS_SUPPORTED_TYPES

Eugene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 10:39:45 PDT 2020


ebrevdo accepted this revision.
ebrevdo added a comment.
This revision is now accepted and ready to land.

This TypeSpec will no longer match specs generated by stringifying tensorflow types.  For example.

  In [4]: tf.int64.name 
  Out[4]: 'int64'

meaning we will have to do the string conversion in python instead of in C++; either way we can't really get away from it.  Your choice where you want us to do it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86549



More information about the llvm-commits mailing list