Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Commit b0c7152

Browse files
authored
Removed reference to TF Lite from kernel selection
1 parent a95a940 commit b0c7152

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

rfcs/20200712-tfrt-kernel-fallback.md

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -539,9 +539,8 @@ helps us cut down on binary size.
539539
540540
## Selecting which kernels to register
541541
542-
We want to add a script to the TF Lite build setup that can determine required
543-
kernels based on a model. We would then only build these kernels. For now, we
544-
will only support selective registration when building from source.
542+
We want to add a script to build configurations that can determine required
543+
kernels based on a model. We would then only build these kernels. For now, we will only support selective registration when building from source.
545544
546545
Script details still need to be worked out.
547546
@@ -552,11 +551,14 @@ Runtime Fallback will call TensorFlow Eager C API (corresponding RFC should be
552551
published soon). Main trade offs between the two fallbacks are described in the
553552
table below:
554553
555-
Property | TFRT Kernel Fallback | TFRT Runtime Fallback
556-
----------- | --------------------- | ---------------------
557-
Generality | Support subset of ops | Support all ops
558-
Performance | Lower overhead | Higher overhead
559-
Binary size | Lower (no TF runtime) | Higher
554+
Property | TFRT Kernel Fallback | TFRT Runtime Fallback
555+
----------- | ---------------------------------------------- | ---------------------
556+
Generality | Support subset of ops (for e.g. no resources*) | Support all ops
557+
Performance | Lower overhead | Higher overhead
558+
Binary size | Lower (no TF runtime) | Higher
559+
560+
\* Long term we might support resources, but we consider them lower priority
561+
due to significant work involved.
560562
561563
### Performance Implications
562564

0 commit comments

Comments
 (0)