This is an archive of the discontinued LLVM Phabricator instance.

[mlgo] Use LLVM_HAVE_TFLITE instead of LLVM_HAVE_TF_API
ClosedPublic

Authored by kazu on Dec 14 2022, 2:45 PM.

Details

Summary

This patch replaces uses of LLVM_HAVE_TF_API with LLVM_HAVE_TFLITE in
a couple of CMakeLists.txt.

Now that 842b0d0fe2dd142305a9461e50cdce9aff7f86bc has landed,
we now have:

LLVM_HAVE_TF_API is defined if and only if LLVM_HAVE_TFLITE
evaluates to true

in the CMake variable world (assuming that you do not set
LLVM_HAVE_TF_API on the cmake invocation).

FWIW, the story is a little different in the C++ macro world, where:

LLVM_HAVE_TF_API is defined if and only if LLVM_HAVE_TFLITE is
defined

This is why edc83a15b45e6b91fce3f35622a6b0a6d34e5211 consisted only of
mechanical replacements.

Diff Detail

Event Timeline

kazu created this revision.Dec 14 2022, 2:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2022, 2:45 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
kazu requested review of this revision.Dec 14 2022, 2:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2022, 2:45 PM
mtrofin accepted this revision.Dec 15 2022, 11:09 AM
This revision is now accepted and ready to land.Dec 15 2022, 11:09 AM
This revision was landed with ongoing or failed builds.Dec 15 2022, 11:11 AM
This revision was automatically updated to reflect the committed changes.