Ticket #1516: premake_flags.diff

build/premake/premake4.lua

@@ -10 +10 @@
 newoption { trigger = "without-nvtt", description = "Disable use of NVTT" }
 newoption { trigger = "without-tests", description = "Disable generation of test projects" }
 newoption { trigger = "without-pch", description = "Disable generation and usage of precompiled headers" }
+newoption { trigger = "minimal-flags", description = "Only set compiler/linker flags that are really needed" }
+newoption { trigger = "disable-preset-flags", description = "Disable all preset compiler/linker flags" }
 newoption { trigger = "with-system-nvtt", description = "Search standard paths for nvidia-texture-tools library, instead of using bundled copy" }
 newoption { trigger = "with-system-enet", description = "Search standard paths for libenet, instead of using bundled copy" }
 newoption { trigger = "with-system-mozjs185", description = "Search standard paths for libmozjs185, instead of using bundled copy" }
@@ -135 +137 @@
 function project_set_build_flags()
 
     flags { "Symbols", "NoEditAndContinue" }
-    if not _OPTIONS["icc"] then
+    if not _OPTIONS["icc"] and not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
         -- adds the -Wall compiler flag
         flags { "ExtraWarnings" } -- this causes far too many warnings/remarks on ICC
     end
@@ -144 +146 @@
         defines { "DEBUG" }
 
     configuration "Release"
-        flags { "OptimizeSpeed" }
+        if not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
+            flags { "OptimizeSpeed" }
+        end
         defines { "NDEBUG", "CONFIG_FINAL=1" }
 
     configuration { }
@@ -176 +180 @@
         flags { "NativeWChar" }
 
     else    -- *nix
-        if _OPTIONS["icc"] then
+        if _OPTIONS["icc"] and not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
             buildoptions {
                 "-w1",
                 -- "-Wabi",
@@ -190 +194 @@
                 "-wd1292" -- avoid lots of 'attribute "__nonnull__" ignored'
             }
             configuration "Debug"
-                buildoptions { "-O0" } -- ICC defaults to -O2
+                if not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
+                    buildoptions { "-O0" } -- ICC defaults to -O2
+                end
             configuration { }
             if os.is("macosx") then
                 linkoptions { "-multiply_defined","suppress" }
             end
         else
-            buildoptions {
-                -- enable most of the standard warnings
-                "-Wno-switch",      -- enumeration value not handled in switch (this is sometimes useful, but results in lots of noise)
-                "-Wno-reorder",     -- order of initialization list in constructors (lots of noise)
-                "-Wno-invalid-offsetof",    -- offsetof on non-POD types (see comment in renderer/PatchRData.cpp)
-
-                "-Wextra",
-                "-Wno-missing-field-initializers",  -- (this is common in external headers we can't fix)
-
-                -- add some other useful warnings that need to be enabled explicitly
-                "-Wunused-parameter",
-                "-Wredundant-decls",    -- (useful for finding some multiply-included header files)
-                -- "-Wformat=2",        -- (useful sometimes, but a bit noisy, so skip it by default)
-                -- "-Wcast-qual",       -- (useful for checking const-correctness, but a bit noisy, so skip it by default)
-                "-Wnon-virtual-dtor",   -- (sometimes noisy but finds real bugs)
-                "-Wundef",              -- (useful for finding macro name typos)
-
-                -- enable security features (stack checking etc) that shouldn't have
-                -- a significant effect on performance and can catch bugs
-                "-fstack-protector-all",
-                "-D_FORTIFY_SOURCE=2",
-
-                -- always enable strict aliasing (useful in debug builds because of the warnings)
-                "-fstrict-aliasing",
-
-                -- do something (?) so that ccache can handle compilation with PCH enabled
-                "-fpch-preprocess",
-
-                -- don't omit frame pointers (for now), because performance will be impacted
-                -- negatively by the way this breaks profilers more than it will be impacted
-                -- positively by the optimisation
-                "-fno-omit-frame-pointer"
-            }
+            -- TODO maybe move some of them out of there
+            if not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
+                buildoptions {
+                    -- enable most of the standard warnings
+                    "-Wno-switch",      -- enumeration value not handled in switch (this is sometimes useful, but results in lots of noise)
+                    "-Wno-reorder",     -- order of initialization list in constructors (lots of noise)
+                    "-Wno-invalid-offsetof",    -- offsetof on non-POD types (see comment in renderer/PatchRData.cpp)
+
+                    "-Wextra",
+                    "-Wno-missing-field-initializers",  -- (this is common in external headers we can't fix)
+
+                    -- add some other useful warnings that need to be enabled explicitly
+                    "-Wunused-parameter",
+                    "-Wredundant-decls",    -- (useful for finding some multiply-included header files)
+                    -- "-Wformat=2",        -- (useful sometimes, but a bit noisy, so skip it by default)
+                    -- "-Wcast-qual",       -- (useful for checking const-correctness, but a bit noisy, so skip it by default)
+                    "-Wnon-virtual-dtor",   -- (sometimes noisy but finds real bugs)
+                    "-Wundef",              -- (useful for finding macro name typos)
+
+                    -- enable security features (stack checking etc) that shouldn't have
+                    -- a significant effect on performance and can catch bugs
+                    "-fstack-protector-all",
+                    "-D_FORTIFY_SOURCE=2",
+
+                    -- always enable strict aliasing (useful in debug builds because of the warnings)
+                    "-fstrict-aliasing",
+
+                    -- do something (?) so that ccache can handle compilation with PCH enabled
+                    "-fpch-preprocess",
+
+                    -- don't omit frame pointers (for now), because performance will be impacted
+                    -- negatively by the way this breaks profilers more than it will be impacted
+                    -- positively by the optimisation
+                    "-fno-omit-frame-pointer"
+                }
+            end
 
-            if arch == "x86" or arch == "amd64" then
+            if (arch == "x86" or arch == "amd64") and not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
                 buildoptions {
                     -- enable SSE intrinsics
                     "-msse"
@@ -245 +254 @@
                 buildoptions { "-mthumb -mcpu=cortex-a9 -mfpu=neon -mfloat-abi=softfp" }
             end
 
-            if os.is("linux") or os.is("bsd") then
+            if (os.is("linux") or os.is("bsd")) and not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
                 linkoptions { "-Wl,--no-undefined", "-Wl,--as-needed" }
             end
 
@@ -256 +265 @@
 
             -- To support intrinsics like __sync_bool_compare_and_swap on x86
             -- we need to set -march to something that supports them
-            if arch == "x86" then
+            if arch == "x86" and not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
                 buildoptions { "-march=i686" }
             end
 
@@ -267 +276 @@
             end
         end
 
-        buildoptions {
-            -- Hide symbols in dynamic shared objects by default, for efficiency and for equivalence with
-            -- Windows - they should be exported explicitly with __attribute__ ((visibility ("default")))
-            "-fvisibility=hidden"
-        }
+        if not (_OPTIONS["minimal-flags"] or _OPTIONS["disable-preset-flags"]) then
+            buildoptions {
+                -- Hide symbols in dynamic shared objects by default, for efficiency and for equivalence with
+                -- Windows - they should be exported explicitly with __attribute__ ((visibility ("default")))
+                "-fvisibility=hidden"
+            }
+        end
 
         -- X11 includes may be installed in one of a gadzillion of three places
         -- Famous last words: "You can't include too much! ;-)"
@@ -292 +303 @@
             defines { "INSTALLED_LIBDIR=" .. _OPTIONS["libdir"] }
         end
 
-        if os.is("linux") or os.is("bsd") then
+        if (os.is("linux") or os.is("bsd")) and not _OPTIONS["with-system-mozjs185"] then
+        -- TODO add "minimal-flags" and/or "disable-preset-flags" here
             -- To use our local SpiderMonkey library, it needs to be part of the
             -- runtime dynamic linker path. Add it with -rpath to make sure it gets found.
             if _OPTIONS["libdir"] then
@@ -841 +853 @@
         -- install_name settings aren't really supported yet by premake, but there are plans for the future.
         -- we currently use this hack to work around some bugs with wrong install_names.
         if target_type == "SharedLib" then
-            linkoptions { "-install_name @executable_path/lib"..project_name..".dylib" }        
+            linkoptions { "-install_name @executable_path/lib"..project_name..".dylib" }
         end
     end
 
@@ -1008 +1020 @@
 
         -- FCollada is not aliasing-safe, so disallow dangerous optimisations
         -- (TODO: It'd be nice to fix FCollada, but that looks hard)
-        buildoptions { "-fno-strict-aliasing" }
+        if not _OPTIONS["disable-preset-flags"] then
+            buildoptions { "-fno-strict-aliasing" }
+        end
 
         buildoptions { "-rdynamic" }
         linkoptions { "-rdynamic" }
@@ -1018 +1032 @@
             links { "c", }
         end
 
-        buildoptions { "-fno-strict-aliasing" }
+        if not _OPTIONS["disable-preset-flags"] then
+            buildoptions { "-fno-strict-aliasing" }
+        end
 
         buildoptions { "-rdynamic" }
         linkoptions { "-rdynamic" }
@@ -1029 +1045 @@
         -- install_name settings aren't really supported yet by premake, but there are plans for the future.
         -- we currently use this hack to work around some bugs with wrong install_names.
         if target_type == "SharedLib" then
-            linkoptions { "-install_name @executable_path/lib"..project_name..".dylib" }        
+            linkoptions { "-install_name @executable_path/lib"..project_name..".dylib" }
         end
 
-        buildoptions { "-fno-strict-aliasing" }
+        if not _OPTIONS["disable-preset-flags"] then
+            buildoptions { "-fno-strict-aliasing" }
+        end
 
         -- On OSX, fcollada uses a few utility functions from coreservices
         linkoptions { "-framework CoreServices" }