Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Xtensa] Add MINMAX/CLAMPS features (LLVM-278) #78

Closed
wants to merge 3 commits into from
Closed

[Xtensa] Add MINMAX/CLAMPS features (LLVM-278) #78

wants to merge 3 commits into from

Conversation

zRedShift
Copy link

  • [Xtensa] Add MINMAX feature
  • [Xtensa] Add CLAMPS feature

@github-actions github-actions bot changed the title [Xtensa] Add MINMAX/CLAMPS features [Xtensa] Add MINMAX/CLAMPS features (LLVM-278) Jul 16, 2023
@zRedShift zRedShift mentioned this pull request Jul 17, 2023
Open
@andreisfr
Copy link
Collaborator

Hi, @zRedShift! Thank you very much for your pull requests. We investigating them and we will prepare update.

@espressif-bot espressif-bot added Status: In Progress work is in progress and removed Status: Opened labels Jul 20, 2023
@zRedShift
Copy link
Author

@andreisfr Sure, let me know if you want me to roll up all my PR's into one for easier checking (like here).

There was another "easy win" in my investigations, the ABS intstruction, hopefully you don't mind me adding it to this PR.

@ramtej
Copy link

ramtej commented Jul 27, 2023

Gentlemen, do we have an idea yet as to when the PR can be merged? Thx!

@espressif-bot espressif-bot added Status: In Review and removed Status: In Progress work is in progress labels Jul 27, 2023
@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA and removed Status: In Review labels Aug 31, 2023
@gerekon
Copy link
Collaborator

gerekon commented Sep 1, 2023

Closed by 26ca9b6 and c81a5b8

@gerekon gerekon closed this Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Resolution: NA Status: Done Issue is done internally
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@zRedShift @andreisfr @ramtej @gerekon @espressif-bot