Skip to content

[Macros] Add a frontend flag -Rmacro-loading to remark on macro resolution #68154

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

Merged
merged 1 commit into from
Aug 26, 2023

Conversation

DougGregor
Copy link
Member

Macro implementations can come from various locations associated with different search paths. Add a frontend flag -Rmacro-loading to emit a remark when each macro implementation module is resolved, providing the kind of macro (shared library, executable, shared library loaded via the plugin server) and appropriate paths. This allows one to tell from the build load which macros are used.

Addresses rdar://110780311.

…olution

Macro implementations can come from various locations associated with
different search paths. Add a frontend flag `-Rmacro-loading` to emit
a remark when each macro implementation module is resolved, providing
the kind of macro (shared library, executable, shared library loaded
via the plugin server) and appropriate paths. This allows one to tell
from the build load which macros are used.

Addresses rdar://110780311.
@DougGregor
Copy link
Member Author

@swift-ci please smoke test

Copy link
Contributor

@bnbarham bnbarham left a comment

Choose a reason for hiding this comment

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

Nice, remarks like this are always handy!

@@ -25,12 +25,26 @@
// RUN: -typecheck -verify \
// RUN: -swift-version 5 -enable-experimental-feature Macros \
// RUN: -external-plugin-path %t/plugins#%swift-plugin-server \
// RUN: -Rmacro-loading -verify-ignore-unknown \
Copy link
Contributor

Choose a reason for hiding this comment

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

Am I just missing where this one is checked? Seems like only macro-loading.txt below is.

@DougGregor DougGregor merged commit 1c5dcb5 into swiftlang:main Aug 26, 2023
@DougGregor DougGregor deleted the remark-macro-loading branch August 26, 2023 02: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.

2 participants