Skip to content

[Driver] Expose -emit-parseable-module-interface[-path] #19727

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

jrose-apple
Copy link
Contributor

@jrose-apple jrose-apple commented Oct 5, 2018

Commit to a command line option spelling so that build systems can start testing it. I deliberately picked one of the longer names we were considering because we can always decide to add a shorter alias, but can't decide a shorter name was too generic. This follows the new name from #19713.

Like the other supplementary output flags, -emit-parseable-module-interface-path will emit a .swiftinterface file to a particular path, while -emit-parseable-module-interface will put it next to the main output (the one specified with -o).

rdar://problem/43776945
rdar://problem/44113493

Commit to a command line option spelling so that build systems can
start testing it. I deliberately picked one of the longer names we
were considering because we can always decide to add a shorter alias,
but can't decide a shorter name was too generic.

Like the other supplementary output flags,
-emit-parseable-module-interface-path will emit a .swiftinterface file
to a particular path, while -emit-parseable-module-interface will put
it next to the main output (the one specified with -o).

rdar://problem/43776945
@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

@jrose-apple jrose-apple force-pushed the emit-parseable-module-interface branch from c38fcc1 to 4ed6fc6 Compare October 5, 2018 01:01
There's no place to put the bridging header in a swiftinterface, and
they don't make sense with the intended use case of distribution.
Just disallow it up front.

rdar://problem/44113493
@jrose-apple jrose-apple force-pushed the emit-parseable-module-interface branch from 4ed6fc6 to 45557ee Compare October 5, 2018 01:02
@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

Copy link
Contributor

@graydon graydon left a comment

Choose a reason for hiding this comment

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

Ok! I thought the name was still up for deciding but I don't care :)

@jrose-apple jrose-apple merged commit 6a1be02 into swiftlang:master Oct 5, 2018
@jrose-apple jrose-apple deleted the emit-parseable-module-interface branch October 5, 2018 18:34
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