Skip to content

Layout left padded - #38

Merged
tpadioleau merged 11 commits into
mainfrom
layout_left_padded
Jun 9, 2026
Merged

Layout left padded#38
tpadioleau merged 11 commits into
mainfrom
layout_left_padded

Conversation

@tim-pearson

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread euler_operators/godunov.hpp Outdated
Comment thread euler_operators/godunov.hpp
@tim-pearson
tim-pearson requested a review from tpadioleau June 2, 2026 14:30
Comment thread euler_operators/godunov.hpp Outdated
Comment thread euler_operators/godunov.hpp Outdated

@tpadioleau tpadioleau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me, did you review the other files ?

Comment thread euler_operators/godunov.hpp Outdated
Co-authored-by: Thomas Padioleau <thomas.padioleau@cea.fr>
@tim-pearson

Copy link
Copy Markdown
Collaborator Author

Looks good to me, did you review the other files ?

Yes I did, looks alright to me.
Maybe I should do one last benchmark for godunov on A100 and SKX to be sure there is no slowdown before merging

@tim-pearson

Copy link
Copy Markdown
Collaborator Author

All good for me, you can merge

@tpadioleau
tpadioleau merged commit b7ce0c6 into main Jun 9, 2026
8 checks passed
@tpadioleau
tpadioleau deleted the layout_left_padded branch June 9, 2026 13:32
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