Skip to content

Update test for https://github.com/apple/swift/pull/74201 #8894

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

Conversation

slavapestov
Copy link

No description provided.

@slavapestov
Copy link
Author

swiftlang/swift#74299
@swift-ci Please test

'[inlined] freestanding macro expansion #1 of stringify',
'13testStringify'
'[inlined] freestanding macro expansion #1 of stringify in module a file main.swift line 5 column 11',
'__swiftmacro_1a0015mainswift_tzEGbfMX4_10_9stringifyfMf_'

Choose a reason for hiding this comment

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

Suggested change
'__swiftmacro_1a0015mainswift_tzEGbfMX4_10_9stringifyfMf_'
'stringify'

This should suffice for what we're interested in here, and should be more future proof.

Choose a reason for hiding this comment

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

@adrian-prantl did you also mean removing the in module a file...?

Choose a reason for hiding this comment

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

no, just the details of the mangled name that are more likely to change

@slavapestov slavapestov force-pushed the macro-expansion-mangling-6.0 branch from 12d5b82 to eb9ac4d Compare June 13, 2024 21:42
@slavapestov
Copy link
Author

swiftlang/swift#74299
@swift-ci Please test

@bnbarham bnbarham merged commit f86beca into swiftlang:swift/release/6.0 Jun 14, 2024
3 checks passed
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