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

glsl-in: relational vector functions improvements #2227

Merged
merged 4 commits into from
Jan 31, 2023

Conversation

JCapucho
Copy link
Collaborator

This PR contains some improvements related to the builtin relational vector functions, it's split into four independent commits that can be reviewed and tested individually.

Changes:

  • Fix equal and notEqual not accepting boolean vectors
  • Fix the relational vector functions not accepting double vectors (these can be used wherever float vectors can be used)
  • Adds tests for these functions
  • Add the not builtin (fixes [glsl-in] "not" is unsupported #2224)

Copy link
Member

@teoxoy teoxoy left a comment

Choose a reason for hiding this comment

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

Nicely done! Needs a rebase since we landed #2229.

The `equal` and `notEqual` builtins also operate on boolean vectors but
the overloads accepting them were not added. This commit fixes that.
All relational vector builtins that operate on floats can also operate
on doubles but these overloads were not present. This commit fixes that.
@JCapucho JCapucho force-pushed the glsl-bool-rel-builtins-fixes branch from f2bca93 to ff5b957 Compare January 31, 2023 13:25
@JCapucho
Copy link
Collaborator Author

@teoxoy thanks for the heads up, since you've already approved I'll set it to auto merge

@JCapucho JCapucho enabled auto-merge (rebase) January 31, 2023 13:28
@JCapucho JCapucho merged commit ae049ed into gfx-rs:master Jan 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[glsl-in] "not" is unsupported
2 participants