amber icon indicating copy to clipboard operation
amber copied to clipboard

dxc_helper.cc compile problems

Open normrubin opened this issue 4 years ago • 4 comments

I'm having some trouble compiling amber with dxc support - dxc_helper.cc pulls in dxcapi.h which for me gets three compile errors

it seems to need dxc/support/WinIncludes.h to get definitions and seems to need functions AddRef and Release defined in class IncludeHandler

I'm trying to figure out if this is my error? does the dxc support get regularly tested? Has anyone else seen these issues?

normrubin avatar Jan 30 '21 14:01 normrubin

@normrubin Thank you for reporting this error. I will quickly check it and get back to you.

jaebaek avatar Feb 02 '21 21:02 jaebaek

I tried

git clone https://github.com/google/amber
cd amber
./tools/git-sync-deps --with-swiftshader --with-dxc
mkdir -p out/TEST
cmake -GNinja -DAMBER_USE_DXC=True -DAMBER_USE_LOCAL_VULKAN ../..
ninja

I did not get any build error. Could you please try the above commands?

jaebaek avatar Feb 02 '21 21:02 jaebaek

i got the failure when compiling on a windows machine dxcapi pulls in different includes on windows vs linux

Sorry I should have said that this was a windows only issue

I do not see any problems building on linux

normrubin avatar Feb 03 '21 13:02 normrubin

I added to the message on the git hub- My issue shows up windows only

On Tue, Feb 2, 2021 at 4:23 PM Jaebaek Seo [email protected] wrote:

I tried

git clone https://github.com/google/amber cd amber ./tools/git-sync-deps --with-swiftshader --with-dxc mkdir -p out/TEST cmake -GNinja -DAMBER_USE_DXC=True -DAMBER_USE_LOCAL_VULKAN ../.. ninja

I did not get any build error. Could you please try the above commands?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/google/amber/issues/942#issuecomment-771997456, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANARWRIZAXSJ3IJV3DHD2LTS5BULJANCNFSM4W2LMQBQ .

normrubin avatar Feb 04 '21 02:02 normrubin