Skip to content
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

Fix compilation #16

Merged
merged 1 commit into from
Aug 25, 2018
Merged

Fix compilation #16

merged 1 commit into from
Aug 25, 2018

Conversation

ccoors
Copy link

@ccoors ccoors commented Aug 24, 2018

I don't know if you accept pull requests and you don't have a license for this project, so in case you need legal assurance I release these two very minor fixes under public domain :)

This contains two compilation fixes:

[ 70%] Building CXX object Source/CMakeFiles/VEZ.dir/VEZ_ext.cpp.o
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp: In member function ‘VkResult vez::Device::BufferSubData(vez::Buffer*, VkDeviceSize, VkDeviceSize, const void*)’:
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:393:70: error: ‘ceil’ is not a member of ‘std’
             auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesToCopy / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                      ^~~~
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:393:70: note: suggested alternative: ‘cerr’
             auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesToCopy / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                      ^~~~
                                                                      cerr
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp: In member function ‘VkResult vez::Device::UncompressedImageSubData(vez::Image*, const VezImageSubDataInfo*, const void*)’:
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:816:82: error: ‘ceil’ is not a member of ‘std’
                         auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesCopied / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                                  ^~~~
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:816:82: note: suggested alternative: ‘cerr’
                         auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesCopied / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                                  ^~~~
                                                                                  cerr
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp: In member function ‘VkResult vez::Device::CompressedImageSubData(vez::Image*, const VezImageSubDataInfo*, const void*)’:
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:956:82: error: ‘ceil’ is not a member of ‘std’
                         auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesCopied / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                                  ^~~~
/home/ccoors/dev/V-EZ/Source/Core/Device.cpp:956:82: note: suggested alternative: ‘cerr’
                         auto alignedBytesToCopy = static_cast<VkDeviceSize>(std::ceil(bytesCopied / static_cast<float>(alignedFlushSize))) * alignedFlushSize;
                                                                                  ^~~~
                                                                                  cerr

and the second one:

[ 44%] Linking CXX shared library ../../Bin/x86_64/libVEZ.so
/usr/bin/ld: ../../Bin/x86_64/libspirv-cross-core.a(spirv_cross.cpp.o): relocation R_X86_64_PC32 against symbol `_ZN11spirv_cross13CompilerErrorD1Ev' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: bad value
collect2: error: ld returned 1 exit status

@soconne soconne merged commit 3970234 into GPUOpen-LibrariesAndSDKs:master Aug 25, 2018
This was referenced Aug 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants