Skip to content

Apply clang-format directives to prevent reformatting of diagrams - #8029

Merged
dsouzai merged 1 commit into
eclipse-omr:masterfrom
hzongaro:fix-up-tree-art
Nov 6, 2025
Merged

Apply clang-format directives to prevent reformatting of diagrams#8029
dsouzai merged 1 commit into
eclipse-omr:masterfrom
hzongaro:fix-up-tree-art

Conversation

@hzongaro

@hzongaro hzongaro commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Diagrams sometimes appear in code comments to illustrate fragments of IL trees, control flow graphs or memory layout. Restore some instances that had been reformatted by clang format, and use the clang-format directive to prevent any future reformatting of those instances.

@hzongaro

hzongaro commented Nov 5, 2025

Copy link
Copy Markdown
Contributor Author

@dsouzai, may I ask you to review this pull request?

@0xdaryl

0xdaryl commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Thanks for finding and fixing these @hzongaro .

@0xdaryl 0xdaryl removed their assignment Nov 5, 2025

@dsouzai dsouzai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There's a couple of minor errors, but overall LGTM

Comment thread compiler/z/codegen/snippet/PPA1Snippet.hpp Outdated
Comment thread compiler/z/codegen/snippet/PPA1Snippet.hpp Outdated
Diagrams sometimes appear in code comments to illustrate fragments of
IL trees, control flow graphs or memory layout.  Restore some instances
that had been reformatted by clang format, and use the clang-format
directive to prevent any future reformatting of those instances.

Signed-off-by:  Henry Zongaro <zongaro@ca.ibm.com>
@dsouzai

dsouzai commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

jenkins build all

@dsouzai
dsouzai merged commit 377a839 into eclipse-omr:master Nov 6, 2025
14 checks passed
@hzongaro
hzongaro deleted the fix-up-tree-art branch November 6, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants