Skip to content

Conversation

@jansvoboda11
Copy link

This PR adopts the new CompilerInvocation::visitPaths() API from upstream to remove some duplication.

@jansvoboda11
Copy link
Author

@swift-ci please test llvm

Copy link

@qiongsiwu qiongsiwu left a comment

Choose a reason for hiding this comment

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

Thanks for the patch!

Copy link

@qiongsiwu qiongsiwu left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my questions. LGTM!

jansvoboda11 added a commit to llvm/llvm-project that referenced this pull request Nov 14, 2025
llvm-sync bot pushed a commit to arm/arm-toolchain that referenced this pull request Nov 14, 2025
@jansvoboda11 jansvoboda11 force-pushed the jan_svoboda/ci-visitpath branch from 9f3356c to a37ebd2 Compare November 14, 2025 02:24
@jansvoboda11
Copy link
Author

@swift-ci please test llvm

1 similar comment
@jansvoboda11
Copy link
Author

@swift-ci please test llvm

@jansvoboda11 jansvoboda11 merged commit c941703 into next Nov 14, 2025
0 of 2 checks passed
@jansvoboda11 jansvoboda11 deleted the jan_svoboda/ci-visitpath branch November 14, 2025 18:19
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.

3 participants