Use -fsystem-module if the compiler is new enough to support it (Xcode 12.5/Swift 5.4 or higher).
#626
Conversation
…ode 12.5/Swift 5.4 or higher). PiperOrigin-RevId: 373046121 (cherry picked from commit 4cd9b2c)
|
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
…ode 12.5/Swift 5.4 or higher). PiperOrigin-RevId: 373046121
| # If enabled, the `-Xcc -fsystem-module` flag will be passed when compiling a | ||
| # system C/Objective-C module (with feature `swift.system_module`) because the | ||
| # compiler is new enough to honor it correctly. If disabled, we attempt to mimic | ||
| # this by disabling certain warnings; however, this unfortunately causes `UInt` | ||
| # APIs to be imported by ClangImporter as `UInt` instead of `Int` because | ||
| # ClangImporter doesn't recognize them as true system modules. |
There was a problem hiding this comment.
That took me so long to figure out 😅
keith
deleted the
ks/use-fsystem-module-if-the-compiler-is-new-enough-to-support-it-xcode-12.5-swift-5.4-or-higher
branch
PiperOrigin-RevId: 373046121
(cherry picked from commit 4cd9b2c)