Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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.

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

Propagate constants more #70

Closed
JulianKemmerer opened this issue Mar 24, 2022 · 2 comments
Closed

Propagate constants more #70

JulianKemmerer opened this issue Mar 24, 2022 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@JulianKemmerer
Copy link
Owner

JulianKemmerer commented Mar 24, 2022

Ex.

point.x = 1;
point.y = 2;
other_point = point;
//return point.x  + point.y; // Does correctly resolve to constant
// Does not resolve constant through 'other_point' struct compound reference
return other_point.x + other_point.y;
// Pseudo code
uint foo(x,y)
{
  return x + y;
}
a = foo(1,2); // Does not resolve to constant

Same for bit manipulation and a bunch of math functions. And finally 'null' all-zeros values too.

@JulianKemmerer JulianKemmerer added the enhancement New feature or request label Mar 24, 2022
@JulianKemmerer JulianKemmerer self-assigned this Mar 24, 2022
@JulianKemmerer
Copy link
Owner Author

Very much related to #65

@JulianKemmerer JulianKemmerer added the help wanted Extra attention is needed label Sep 1, 2022
@JulianKemmerer
Copy link
Owner Author

As of closing #65

The constant value of the expression is not resolved. But it is known that things downstream/derived from it are constant too (not part of pipeline registers,etc)

point.x = 1;
point.y = 2;
other_point = point;
//return point.x  + point.y; // Does correctly resolve to constant
// Does not resolve constant through 'other_point' struct compound reference
return other_point.x + other_point.y;
^ As of Issue #65 close 'other_point.x + other_point.y' is known to be constant (value not known though)

@JulianKemmerer JulianKemmerer removed their assignment Feb 3, 2023
Repository owner locked and limited conversation to collaborators Nov 16, 2024
@JulianKemmerer JulianKemmerer converted this issue into discussion #224 Nov 16, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant