-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Windows build fails with clang assertion failure #8
Comments
@bejado have you run into this on Windows? The error occurs in clang's internal files, seems like a bug in the toolchain. |
Yes, seems to be an bug in Clang. I believe I remember running into this issue on Windows when using Clang 7, so I switched to Clang 6. @jdarpinian, can you confirm which version you’re using? |
I am using 7 because that's the version recommended in the README. I will try 6 instead. |
same problems with jdarpinian, i tried llvm6 but same. |
Clang 6 works, although it produces tens of thousands of warnings and the binaries crash because they can't find the assets directory. |
We'll try and fix the warnings. Issues have been reported about our Path class not working properly on Windows, that's most likely what you're running into. We'll also fix those. Thanks! |
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
Enhance Animator to allow reusable animation (google#8) - allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset Enhance morpher: add CPU morpher (google#7) - add cpu morpher which supports more than 4 active targets - add common interface for morphers - change Animator to allow swapping out the stock MorphHelper for the new CPUMorpher - change Animator to allow morphers to be managed externally
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
- allow an Animator to be created explictly with an animation asset - such an Animator is "empty" as it contain animation data but is not hooked up to any entities - call addAnimatedAsset to apply the animation to a model asset
Following the build steps, when compiling sample-app.vcxproj (FilamentApp.cpp, IBL.cpp, IcoSphere.cpp, Image.cpp, and NativeWindowHelperWindows.cpp) I get a crash in clang:
Assertion failed: CXXInfo->BaseOffsets.count(Base) && "Did not find base!", file C:\src\llvm_package_336178\llvm\tools\clang\include\clang/AST/RecordLayout.h, line 225
It crashes both with the LLVM version linked in the README and the latest available.
The text was updated successfully, but these errors were encountered: