From e2c2c220c642cc5f1c622f909d0ddfd22e6c04d4 Mon Sep 17 00:00:00 2001 From: Tim Foley Date: Fri, 11 May 2018 16:34:19 -0700 Subject: Generate Visual Studio projects using Premake (#557) * Generate Visual Studio projects using Premake This change adds a `premake5.lua` file that allows us to generate our Visual Studio solution using Premake 5 (https://premake.github.io/). The existing Visual Studio solution/projects are now replaced with the Premake-generated ones, and project contributors will be expected to update these by running premake after adding/removing files. I have *not* changed the Linux `Makefile` build at all, because that file is also used for things like running our tests, so that clobbering it with a premake-generated `Makefile` would break our continuous testing. Hopefully future changes can switch to a generated `Makefile` and perhaps even add an XCode project as well. Notes: * The `build/slang-build.props` file is no longer needed/used, so it has been removed. * The `slang-eval-test` test fixture wasn't following our naming conventions for its directory path, so it was updated to streamline the Premake build configuration work. This required changes to the `Makefile` as well * Some seemingly unncessary preprocessor definitions that were specified for `core` and `slang-glslang` have been dropped. We will see if anything breaks from that. * Possible fixup for Premake vpath issue Premake's `vpath` feature seems to be nondeterministic about the order it applies filters (because Lua isn't deterministic about the order of entries in a key/value table), and as a result we can end up in a weird case where it decides that a `foo.cpp.h` file matches the `**.cpp` filter (I'm not sure why) before it tests against the `**.h` filter. This change uses an (undocumented) Premake facility to set `vpath` using a list of singleton tables, which seems to fix the order in which things get tested. * Remove support for "single-file" build of Slang The `hello` example was the only bit of code that uses the "single-file" way of building Slang, and this had already run up against limitations of the Visual Studio compilers in its Debug|x64 build. Rather than mess with Premake to make it pass through the `/bigobj` linker flag that is needed to work around the issue, it makes more sense just to stop using/supporting the feature since we wouldn't want users to depend on it anyway (our documentation no longer refers to it). While I was at it I went ahead and made sure that the `SLANG_DYNAMIC` flag doesn't need to be set manually, so that instead there is a non-default `SLANG_STATIC` option (not that we have a static-library build of Slang at the moment). --- source/slangc/main.cpp | 1 - source/slangc/slangc.vcxproj | 99 +++++++++++++++++------------------- source/slangc/slangc.vcxproj.filters | 13 +---- 3 files changed, 50 insertions(+), 63 deletions(-) (limited to 'source/slangc') diff --git a/source/slangc/main.cpp b/source/slangc/main.cpp index 391642c3e..376b75212 100644 --- a/source/slangc/main.cpp +++ b/source/slangc/main.cpp @@ -1,6 +1,5 @@ // main.cpp -#define SLANG_DYNAMIC #include "../../slang.h" SLANG_API void spSetCommandLineCompilerMode(SlangCompileRequest* request); diff --git a/source/slangc/slangc.vcxproj b/source/slangc/slangc.vcxproj index 4855b7373..77a7dcea5 100644 --- a/source/slangc/slangc.vcxproj +++ b/source/slangc/slangc.vcxproj @@ -20,81 +20,87 @@ {D56CBCEB-1EB5-4CA8-AEC4-48EA35ED61C7} + true Win32Proj slangc - slangc - 8.1 Application true - v140 Unicode + v140 Application true - v140 Unicode + v140 Application false - v140 - true Unicode + v140 Application false - v140 - true Unicode + v140 - - + - - - + - true + ..\..\bin\windows-x86\debug\ + ..\..\intermediate\windows-x86\debug\slangc\ + slangc + .exe true + ..\..\bin\windows-x64\debug\ + ..\..\intermediate\windows-x64\debug\slangc\ + slangc + .exe false + ..\..\bin\windows-x86\release\ + ..\..\intermediate\windows-x86\release\slangc\ + slangc + .exe false + ..\..\bin\windows-x64\release\ + ..\..\intermediate\windows-x64\release\slangc\ + slangc + .exe - - - Level4 + NotUsing + Level3 + _DEBUG;%(PreprocessorDefinitions) + EditAndContinue Disabled - WIN32;_DEBUG;_CONSOLE;%(PreprocessorDefinitions) MultiThreadedDebug - ../ - false - true Console @@ -103,15 +109,12 @@ - - - Level4 + NotUsing + Level3 + _DEBUG;%(PreprocessorDefinitions) + EditAndContinue Disabled - WIN32;_DEBUG;_CONSOLE;%(PreprocessorDefinitions) MultiThreadedDebug - ../ - false - true Console @@ -120,57 +123,51 @@ - Level4 - - - MaxSpeed + NotUsing + Level3 + NDEBUG;%(PreprocessorDefinitions) + Full true true - WIN32;NDEBUG;_CONSOLE;%(PreprocessorDefinitions) + false + true MultiThreaded - ../ - false - true Console - true true true - Level4 - - - MaxSpeed + NotUsing + Level3 + NDEBUG;%(PreprocessorDefinitions) + Full true true - WIN32;NDEBUG;_CONSOLE;%(PreprocessorDefinitions) + false + true MultiThreaded - ../ - false - true Console - true true true + + + - {f9be7957-8399-899e-0c49-e714fddd4b65} + {F9BE7957-8399-899E-0C49-E714FDDD4B65} - {db00da62-0533-4afd-b59f-a67d5b3a0808} + {DB00DA62-0533-4AFD-B59F-A67D5B3A0808} - - - diff --git a/source/slangc/slangc.vcxproj.filters b/source/slangc/slangc.vcxproj.filters index 0d8d9e457..e9ae1c092 100644 --- a/source/slangc/slangc.vcxproj.filters +++ b/source/slangc/slangc.vcxproj.filters @@ -1,17 +1,8 @@ - + - {4FC737F1-C7A5-4376-A066-2A32D752A2FF} - cpp;c;cc;cxx;def;odl;idl;hpj;bat;asm;asmx - - - {93995380-89BD-4b04-88EB-625FBE52EBFB} - h;hh;hpp;hxx;hm;inl;inc;xsd - - - {67DA6AB6-F800-4c08-8B7A-83BB121AAD01} - rc;ico;cur;bmp;dlg;rc2;rct;bin;rgs;gif;jpg;jpeg;jpe;resx;tiff;tif;png;wav;mfcribbon-ms + {E9C7FDCE-D52A-8D73-7EB0-C5296AF258F6} -- cgit v1.2.3