Skip to content

Conversation

@jakeegan
Copy link
Member

@jakeegan jakeegan commented Apr 1, 2025

No description provided.

Copy link
Collaborator

@hubert-reinterpretcast hubert-reinterpretcast left a comment

Choose a reason for hiding this comment

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

There is at least one "sibling" preprocessor conditional that is not indented the same way at

.

Perhaps the patch should just clang-format the entire file?

@jakeegan jakeegan changed the title [sanitizer_common][NFC] Fix some preprocessor directive formatting [sanitizer_common][NFC] Fix sanitizer_symbolizer_libcdep.cpp formatting Apr 1, 2025
Copy link
Collaborator

@hubert-reinterpretcast hubert-reinterpretcast left a comment

Choose a reason for hiding this comment

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

LGTM!

@jakeegan jakeegan merged commit 50fe5b9 into llvm:main Apr 3, 2025
10 checks passed
@jakeegan jakeegan deleted the sanitizer_symbolizer_libcdep_formatting branch April 3, 2025 14:39
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