Skip to content

Conversation

@DougGregor
Copy link
Member

No description provided.

@DougGregor
Copy link
Member Author

@swift-ci please smoke test

3 similar comments
@DougGregor
Copy link
Member Author

@swift-ci please smoke test

@DougGregor
Copy link
Member Author

@swift-ci please smoke test

@DougGregor
Copy link
Member Author

@swift-ci please smoke test

@DougGregor
Copy link
Member Author

@swift-ci please smoke test

/// case standard
/// }
/// }
@attached(member)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
@attached(member)
@attached(member, names: named(RawValue), named(rawValue), named(`init`), arbitrary)

@DougGregor DougGregor merged commit 2979ed5 into swiftlang:main Mar 4, 2023
@DougGregor DougGregor deleted the option-set-macro branch March 4, 2023 08:37
@etcwilde
Copy link
Member

etcwilde commented Mar 5, 2023

@swift-ci please test

bnbarham added a commit to bnbarham/swift that referenced this pull request Mar 5, 2023
…-macro"

This reverts commit 2979ed5, reversing
changes made to 1c86433.

Causes failures on `validation-test/SIL/parse_stdlib.sil` when parsing
`OptionSet.init`, presumably because name lookup now finds the macro as
well:
```
%10 = witness_method $Self, #OptionSet.init!allocator : <Self where Self : OptionSet> (Self.Type) -> (Self.RawValue) -> Self : $@convention(witness_method: OptionSet) <τ_0_0 where τ_0_0 : OptionSet> (@in τ_0_0.RawValue, @thick τ_0_0.Type) -> @out τ_0_0 // user: %11
```
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