Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@zanderso
Copy link
Member

Reverts #23986

From the prod builders:

FAILED: clang_x64/gen_snapshot 
../../buildtools/mac-x64/clang/bin/clang++ -isysroot /opt/s/w/ir/cache/osx_sdk/XCode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk -mmacosx-version-min=10.11.0 -flto -rdynamic -arch x86_64 -nostdlib++ -stdlib=libc++ -Wl,-dead_strip -Wl,-search_paths_first -L. -Wl,-rpath,@loader_path/. -Wl,-rpath,@loader_path/../../.. -Wl,-pie  -Xlinker -rpath -Xlinker @executable_path/Frameworks -o clang_x64/gen_snapshot -Wl,-filelist,clang_x64/gen_snapshot.rsp  -ldl -lpthread -framework CoreFoundation -framework CoreServices
ld: reference to bitcode symbol '___gxx_personality_v0' which LTO has not compiled in '___gxx_personality_v0' for architecture x86_64
clang-12: error: linker command failed with exit code 1 (use -v to see invocation)
[3829/3842] SOLINK ./libflutter.so
ninja: build stopped: subcommand failed.

@zanderso
Copy link
Member Author

This is a bit confusing because this buildroot roll passed the presubmit checks, but it looks like it caused build failures on the prod builder. @godofredoc are you aware of any configurations differences between the machines?

@godofredoc
Copy link
Contributor

godofredoc commented Jan 28, 2021

This is a bit confusing because this buildroot roll passed the presubmit checks, but it looks like it caused build failures on the prod builder. @godofredoc are you aware of any configurations differences between the machines?

Is it possible that the build failing on prod was filtered out from the PR? all the mac bots have the same specs and provisioned with the same configs.

@godofredoc
Copy link
Contributor

Here: https://github.com/flutter/engine/pull/23986/checks

"Mac Android AOT Engine" passed the presubmit check here https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20Android%20AOT%20Engine/13901/overview

but failed in postsubmit here: https://ci.chromium.org/ui/p/flutter/builders/prod/Mac%20Android%20AOT%20Engine/8662/overview

Took a look at the two builds the configurations are exactly the same. Not sure if having having some intermediate objects in cache can cause this behavior. Prod builds usually have a higher cache hit rate than presubmit builds.

@zanderso
Copy link
Member Author

In the interests of getting the tree open again, I'm going to land this revert, but we will need to ask @godofredoc for help if the potential caching issues happen again on the reland, and we can dig in a bit more then during the day.

@zanderso zanderso merged commit c4d1058 into master Jan 28, 2021
@zanderso zanderso deleted the revert-23986-libcxx-visibility-roll branch January 28, 2021 07:23
@chinmaygarde
Copy link
Member

The linked patch does make it so that the executable will be more diligent about not leaking libc++ symbols. Maybe we were just getting away with it earlier? We should try to reproduce LTO with bitcode builds on Mac with that commit patched it. I suspect it may not be an issue with cached contents.

@stuartmorgan-g
Copy link
Contributor

Was it just Android that failed, or all platforms?

@zanderso
Copy link
Member Author

Looks like mac building for both Android and iOS, but check the red cells on the dashboard on the roll commit for the definitive answer.

@dnfield
Copy link
Contributor

dnfield commented Mar 4, 2021

I believe try builders do not use LTO.

To reproduce this locally, you'll want to build with LTO and with bitcode enabled, which unfortunately means no goma.

hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants