-
Notifications
You must be signed in to change notification settings - Fork 261
do not package Clang internal headers in icecc-create-env #473
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
Conversation
It doesn't seem to be actually needed (tested with Clang 8.0.0 and 4.0.1). I have no idea why I added it, the commit doesn't say. Possibly initially -frewrite-includes had a bug that necessitated this.
|
Looks good to me Qt builds fine with this minimal toolchain as long as I explicitly include the needed headers as described in #471 |
|
Minor note: This is using the environment created by icecc-create-env from this PR. So clang headers where not packaged in the enviornment. The issue disappeared when i used the same icecc environment, but updated my xcode to 10.2. I don't know if this is a discrepancy in the Xcode version <-> swift-provided clang compiler, or something to do with the older version of Xcode. So mostly just some fear of breaking icecc-create-env when using older xcode versions. |
|
Can you be more specific about the problems? E.g. provide a specific error this triggers? I can scratch this change if it turns out it is needed after all, so I'd like to investigate this more. |
|
Here's the build log of the failure while building Qt 5.14 qtbase repo. The circumstances: to use just the cross compiler environment which was built using https://github.com/torarnv/icecc-create-x86_64-env and your PR. So the cross compiler is ran on various Linux machines in our network. Keeping everything the same, but selecting Xcode 10.2.1 instead of 10.1, then the build succeeds. My suspicion is that it could be a discrepancy between Xcode 10.1 and swift-llvm-5.0.1. |
Yes, I don't think those issues were due to the lack of headers in the env. You can probably verify this by doing the same setup but with the headers present, i.e. Xcode 10.1 + swift-llvm-5.0.1-based env with headers. |
|
Can you please confirm whether the problem goes away if you use icecc-create-env without this change? It is not clear from your description, you only mention the problem going away when switching to 10.2, but that's irrelevant here. |
|
Without the PR, Xcode 10.1, swift-llvm-5.0.1 with intrinsic headers, it still fails with the same errors not finding intrinsic functions. So I guess false alarm from my side. |
It doesn't seem to be actually needed (tested with Clang 8.0.0 and 4.0.1).
I have no idea why I added it, the commit doesn't say. Possibly
initially -frewrite-includes had a bug that necessitated this.