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.

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

llvm: Tweaks to structure layout and codestyle #2994

Merged
merged 4 commits into from
Jul 5, 2024

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Jul 5, 2024

Make MT state size 8B aligned.
Make Philox state size 8B aligned.
Use derived types in operations where the expected type is known.
Codestyle.

jvesely added 4 commits July 4, 2024 22:18
Reduce size to keep the size 8B aligned when using fp64.
Use non-legacy get_state() for better control of initializer values.
Codestyle.

Signed-off-by: Jan Vesely <[email protected]>
The size of the structure is 8B aligned.

Signed-off-by: Jan Vesely <[email protected]>
If one of the operands in binary ops and comparisons is known
and the other one is a constant use the known type instead of
ctx.int32_ty to instantiate the constant.

Signed-off-by: Jan Vesely <[email protected]>
Use zero constant if available.
Use up to 120 characteers per line.

Signed-off-by: Jan Vesely <[email protected]>
@jvesely jvesely added the compiler Runtime Compiler label Jul 5, 2024
Copy link

github-actions bot commented Jul 5, 2024

This PR causes the following changes to the html docs (ubuntu-latest-3.11):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely changed the title llvm: Tweaks to structure layout and codestyl llvm: Tweaks to structure layout and codestyle Jul 5, 2024
@jvesely jvesely merged commit 8a0115a into PrincetonUniversity:devel Jul 5, 2024
36 checks passed
@jvesely jvesely deleted the devel-llvm-cleanup branch July 5, 2024 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler Runtime Compiler
Projects
Development

Successfully merging this pull request may close these issues.

1 participant