-
Notifications
You must be signed in to change notification settings - Fork 1.5k
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
error: 'operator delete[]' is unavailable: introduced in macOS 10.12 #870
Comments
Please attach config.log that is produced by configure on your system. |
Sure thing:
|
Thanks. Can you try building with: make CXXFLAGS='-O2 -fno-sized-deallocation' and reporting if it fixes the issue ? |
Compilation succeeded, but linking failed:
EDIT: At first it was giving me an undefined reference to some |
If you do make clean and then repeat my instructions, I think it'll work. |
@alk see my edit :) Caught that right after I posted, haha. |
Ok. So _memalign is something else that we'll need fix as well. As for original issue, can you try this patch (on top of master branch):
I think it should catch the "issue" of osx compiler allowing -fsized-deallocation yet then barking on it's actual use. (You will need to run ./autogen.sh) |
OSX and windows have issues with memalign. So test against tc_memalign instead. This should fix _memalign linker-time part of issue #870.
I've uploaded fix for memalign issue. Thanks for catching it. |
No problem! Do you want me to still test that patch? If so, could you wrap it in a code block? |
Yes, please test it. And yes I've already wrapped it up. |
$ ./autogen.sh && ./configure && make -j8 && echo $?
0 Looks like that worked after pulling master (5618ef7) |
Weirdly, some OSX compiler does allow -fsized-deallocation, yet barks on any attempt to actually use it at compile time (!). So lets detect this as well in configure and opt out as necessary. Should fix issue #870.
Thanks a lot. I've uploaded a fix and we should be done here. |
Can't seem to find anything on the web about this and I've surely never seen this error before.
When building src/tcmalloc.cc I get the following output from make:
I'm on macOS 10.11.6 using clang (I believe
g++
just gets redirected toclang++
, but I've even configured withCXX=clang++ CC=clang
with the same result).The text was updated successfully, but these errors were encountered: