-
Notifications
You must be signed in to change notification settings - Fork 1
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
clang-13 prints many warnings/errors #239
Comments
@yurivict Finally got around to fixing some of these errors. If you have time, could you try with the current repo and let me know the results? |
The
|
@yurivict Does adding the the following line fix the problem?
|
array-related failures are gone, but there are still warnings/errors:
|
@yurivict I've made changes to the source to handle most of the warning/errors. They're on the
There are probably still problems with
Do you know how to implement this for the clang-13 compiler? |
No. |
What does this code achieve? How are regular containers insufficient? |
STL vector generally can't be mmapable because it can generally resize any time. |
The project fails to build with gcc-12 as well: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=266189 |
Thanks for the information, the gcc-12 fail looks like it has to do with the cdlParser.c which is bison output. As for the mmap, I think I was trying to reduce the real memory required by essentially forcing the data to disk. I think the array sizes were fixed when allocated. |
Yes, it's never a good idea to use STL container inner structures, because there are many implementations of STL. Having a custom class for mmapped vector seems like the way to go. Currently the cvc's FreeBSD port doesn't build with clang, and it also prevents GCC upgrade to 12 because it fails with gcc-12. |
FYI |
These warnings likely indicate real errors:
There are also these errors:
The text was updated successfully, but these errors were encountered: