Skip to content

Conversation

RiverDave
Copy link
Collaborator

No description provided.

Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

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

Awesome! LGTM with one minor nit: add a CGOG check on the tests for the original codegen output (you can find examples in upstream and in some tests in the incubator)

@RiverDave RiverDave force-pushed the dave/lower-blend-avx branch from 6581a28 to 629f32c Compare August 26, 2025 00:37
@RiverDave
Copy link
Collaborator Author

Awesome! LGTM with one minor nit: add a CGOG check on the tests for the original codegen output (you can find examples in upstream and in some tests in the incubator)

Gotcha, will keep adding these checks from now on.

@bcardosolopes bcardosolopes merged commit 9b5498a into llvm:main Aug 26, 2025
8 of 9 checks passed
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