-
-
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
Linux: not used because `__STDC_CONSTANT_MACROS' is defined [-Winvalid-pch] #1818
Comments
This seems like this issue - http://stackoverflow.com/questions/986426/what-do-stdc-limit-macros-and-stdc-constant-macros-mean - wonder if defining -std=c11 would fix this. I notice you are using an alien package any reason why? |
The alien package is a multilib equivalent of the standard Slackware gcc made by one of the Slackware maintainers (Alienbob) which I use to build programs like wine or pcsx2. It should not make a difference here. I tried defining -std=c11 for both -DCMAKE_C_FLAGS and -DCMAKE_CXX_FLAGS. The latter predictably causes a new warning while the former does not make much of a difference.
So I then tried to define -std=c++11 for -DCMAKE_CXX_FLAGS and found that it did not make a difference either. |
Also need update submodule GLSLang because missing header in Vulkan/glslang/glslang/MachineIndependent/intermOut.cpp:52:10: error: ‘__builtin_isinf_sign’ is not a member of ‘std’ Missing #include cmath in that file. In the actual code in github KhronosGroup/GLSLang are fixed 👍 |
Thanks for pointing this out @maximu Feel free to send out pull to update the glslang submodule . |
the main problem for the actual glslang code is new dependences so it's problematic linker/path cmake error under some OS need studing before pull request. Also first need fix Acces violation reading location and rpcs3 can't open
Maybe I miss in some part ??? |
Well I detected that error reported by libpng16 static with Libpng in my system running ok... We maybe remove libpng static under linux beucase break relocations problem. |
I'm update in my repository https://github.com/maximu/rpcs3 |
@GhettoGirl @mrcmunir It would be nice if both your work got into master. I've started implementing Vulkan under Linux https://github.com/rcaridade145/rpcs3/commit/3a4923b9419b288b5da4bcd669ab4328a9ec9a08 and i believe your work would unlock my problems. |
@rcaridade145 Thanks I detected some problem Gtk-ERROR **: GTK+ 2.x symbols detected. Using GTK+ 2.x and GTK+ 3 in the same process is not supported Won't run on Mate Desktop because they no ported GUI Desktop to GTK3 so maybe broken for Mate desktop. |
@mrcmunir RPCS3 uses wxWidgets which has several backends (Qt5, GTK2, GTK3, Motif, X11, OpenGL, DirectFB, MS-Windows ( OFF TOPIC: I personally prefer Qt5 as backend - I hate GTK so much. |
Why not just statically link on the problematic platforms? |
@mrcmunir I didn't check yet for different versions of GTK on Ubuntu it is using GTK3 so i concentrated on that first. To get it to work on GTK2 one would need to adapt the code to consider this - http://stackoverflow.com/questions/14788439/getting-x11-window-handle-from-gtkwidget . Then detect on CMakeList the correct GTK or backend of WxWidgets. |
Well I solved that problem force wxwidgets rebuild with gtk3.
The main problem is by default wxgtk package under archlinux install with gtk2 and not for gtk3 support. |
You can always rebuild packages in arch using the PKGBUILD and editing the build flags, wxgtk does not take that long to compile. Personally I like that I can use gtk2 with rpcs3 which supports my theme best. |
Yes I know with https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=wxgtk-git Change --with-gtk=2 to --with-gtk-3 . This alternative solution for now because GTK2 and GTK3 linker very problematic and incompatibility. Well Finish compile with initial vulkan work @rcaridade145 (But untested because intel won't work anyway limited support in the actual state fail createinstance() Undefined trap. LDR: N3fmt9exceptionE thrown: createInstance(): Undefined trap All changes I updated in my repository https://github.com/mrcmunir/rpcs3/ NEED TESTING . I hope this helps someone to go with vulkan port to linux . |
Great work @mrcmunir . I'll have sometime this weekend to work on this. However that error can be found in this comment #1736 (comment) . I'll try and make the backend code more resilient so we can use gtk2,gtk3, QT5,etc... |
@mrcmunir Concerning the original problem - https://tombarta.wordpress.com/2009/12/12/precompiled-headers-with-gcc/ |
You need Mesa 12 for complete Intel Vulkan support. Also make sure that you enabled the |
@mrcmunir @GhettoGirl I worked a bit this weekend on it but all i could get is a blank window..maybe something i'm missing.. |
@rcaridade145 Do you have the |
@rcaridade145 with that hw running? If you intel HW will needed Gen8 Broadwell GPU and higher Skylake architectures for now under linux because they are missing implementations under Hasswell and Ivy bridge because SIMD running different in that architecture. For enable DRI3 create file in xorg /etc/X11/xorg.conf.d/20-intel.conf
|
@mrcmunir NVIDIA blob |
In that case it should work correctly NVIDIA vulkan samples running very well under linux |
@rcaridade145 have you tried building like it was posted on #1752 ? |
@Nezarn i have. |
Its been a while since I last rebuilt rpcs3 and now that I tried again I can't reproduce these warnings so I am going to close this issue. Thanks! |
this issue appears again with GCC 8.1 i have this tons of this warnings cc1plus: warning: /tmp/makepkg/sl1-rpcs3-git/src/build/rpcs3/cotire/rpcs3_CXX_prefix.hxx.gch: not used because `__STDC_CONSTANT_MACROS' is defined [-Winvalid-pch] |
OS:
Slackware64-current
gcc-5.3.0_multilib-x86_64-3alien
llvm-3.8.0-x86_64-2
Although rpcs3 does compile, the entire build process is warning about
not used because __STDC_CONSTANT_MACROS is defined [-Winvalid-pch]
. Is there anything that can be done to solve these warnings?Full log - http://ks392457.kimsufi.com/orbea/stuff/logs/rpcs3-build.log
The text was updated successfully, but these errors were encountered: