Skip to content

#37464: Update unary LLK Tile API's#37703

Draft
mouliraj-mcw wants to merge 1 commit intomainfrom
mouliraj/unary_llk_changes
Draft

#37464: Update unary LLK Tile API's#37703
mouliraj-mcw wants to merge 1 commit intomainfrom
mouliraj/unary_llk_changes

Conversation

@mouliraj-mcw
Copy link
Contributor

@mouliraj-mcw mouliraj-mcw commented Feb 12, 2026

Ticket

Link to Github Issue #37464

What's changed

Updated the LLK unary tiles so that each API uses a single tile api for different dtypes

Checklist

  • All post-commit tests
  • Blackhole Post commit
  • cpp-unit-tests

@mouliraj-mcw mouliraj-mcw force-pushed the mouliraj/unary_llk_changes branch from 42876a0 to ebbcfd0 Compare February 12, 2026 05:15
@mouliraj-mcw mouliraj-mcw force-pushed the mouliraj/unary_llk_changes branch 2 times, most recently from 36f1c69 to e18c5d2 Compare February 13, 2026 03:32
}
if (input_dtype == DataType::UINT16) {
return {"logical_not_unary_tile_init();", fmt::format("logical_not_unary_tile_uint16({});", idst)};
data_format = "Int32";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

switch/case?

@@ -1,4 +1,4 @@
// SPDX-FileCopyrightText: © 2023 Tenstorrent AI ULC
// SPDX-FileCopyrightText: © 2026 Tenstorrent AI ULC
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't change copyright years when changing files.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assumed that renaming the file would be treated as a delete and recreate, so I updated the copyright year. I’ll revert that change.

@mouliraj-mcw mouliraj-mcw force-pushed the mouliraj/unary_llk_changes branch from e18c5d2 to f9b8763 Compare February 16, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants