Skip to content

Conversation

ilovepi
Copy link
Contributor

@ilovepi ilovepi commented Jun 11, 2025

After c03f377, the TFL dialect is no longer part of common dialects, and tf-tosa-opt --tofl-to-tosa-pipeline would no longer be able to convert from TFL since the dialect was no longer available.

After c03f377, the TFL dialect is no longer part of common dialects, and `tf-tosa-opt --tofl-to-tosa-pipeline` would no longer be able to convert from TFL  since the dialect was no longer available.
@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Jun 11, 2025
@ilovepi
Copy link
Contributor Author

ilovepi commented Jun 11, 2025

FWIW, I do not understand how tests that use --tfl-to-tosa-pipeline option currently pass without that dialect being registered.

Locally I'm seeing the following error on main w/o this patch:

bazel-bin/tensorflow/compiler/mlir/tosa/tf-tosa-opt --tfl-to-tosa-pipeline ~/emitc/model.mlir -o ~/emitc/model_tosa.mlir
/usr/local/google/home/paulkirth/emitc/model.mlir:3:28: error: operation being parsed with an unregistered dialect. If this is intended, please use -allow-unregistered-dialect with the MLIR tool used
    %0 = "tfl.pseudo_const"() <{value = dense<1> : tensor<2xi32>}> : () -> tensor<2xi32>

@ilovepi
Copy link
Contributor Author

ilovepi commented Jun 11, 2025

cc: @mtrofin

@keerthanakadiri keerthanakadiri added the comp:lite-tosa TFLite TOSA conversion issues label Jun 12, 2025
@github-project-automation github-project-automation bot moved this to Assigned Reviewer in PR Queue Jun 12, 2025
@keerthanakadiri keerthanakadiri requested a review from mtrofin June 12, 2025 05:16
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Jun 12, 2025
@mtrofin mtrofin requested review from jpienaar and sjarus June 13, 2025 04:06
@github-project-automation github-project-automation bot moved this from Assigned Reviewer to Approved by Reviewer in PR Queue Jun 13, 2025
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jun 13, 2025
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 13, 2025
@keerthanakadiri keerthanakadiri removed the awaiting review Pull request awaiting review label Jun 16, 2025
@sjarus
Copy link
Collaborator

sjarus commented Jun 16, 2025

@jpienaar is this ok to land ? If so, I can proceed.

@google-ml-butler google-ml-butler bot added the kokoro:force-run Tests on submitted change label Jun 18, 2025
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 18, 2025
@jpienaar jpienaar merged commit 4c0a2a3 into tensorflow:master Jun 18, 2025
2 of 5 checks passed
@github-project-automation github-project-automation bot moved this from Approved by Reviewer to Merged in PR Queue Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:lite-tosa TFLite TOSA conversion issues ready to pull PR ready for merge process size:XS CL Change Size: Extra Small
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

7 participants