-
Notifications
You must be signed in to change notification settings - Fork 611
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
ubsan bot needs rtti? #1174
Comments
Actually I'm not sure. This: suggests adding -fno-rtti. Which we already do. I'll try locally and figure out. |
It's not rtti. Guess we need to use C++ compiler even for C tests. Trying. |
I'm out of ideas. Couldn't get it to work. |
behdad
added a commit
that referenced
this issue
Sep 24, 2018
Ok, making test/api dir use C++ linker to link C code fixes this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://circleci.com/gh/harfbuzz/harfbuzz/33467
Let's build it with stdc++.
The text was updated successfully, but these errors were encountered: