improve has_flags for gcc

3 files changed, 124 insertions, 81 deletions

diff --git a/xmake/modules/detect/flags/has_fsanitize_address.lua b/xmake/modules/detect/flags/has_fsanitize_address.lua
deleted file mode 100644
index 36ef766d4..000000000
--- a/xmake/modules/detect/flags/has_fsanitize_address.lua
+++ /dev/null
@@ -1,34 +0,0 @@
---!The Make-like Build Utility based on Lua
---
--- Licensed to the Apache Software Foundation (ASF) under one
--- or more contributor license agreements.  See the NOTICE file
--- distributed with this work for additional information
--- regarding copyright ownership.  The ASF licenses this file
--- to you under the Apache License, Version 2.0 (the
--- "License"); you may not use this file except in compliance
--- with the License.  You may obtain a copy of the License at
---
---     http://www.apache.org/licenses/LICENSE-2.0
---
--- Unless required by applicable law or agreed to in writing, software
--- distributed under the License is distributed on an "AS IS" BASIS,
--- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
--- See the License for the specific language governing permissions and
--- limitations under the License.
--- 
--- Copyright (C) 2015 - 2017, TBOOX Open Source Group.
---
--- @author      ruki
--- @file        has_fsanitize_address.lua
---
-
--- has flag: '-fsanitize=address'? 
---
--- @param flag  the flag 
--- @param opt   the argument options, .e.g {tool = {name = "", program = "", version = ""}}
---
--- @return      true or false
---
-function main(flag, opt)
-    -- TODO
-end
diff --git a/xmake/modules/detect/tools/gcc/has_flag.lua b/xmake/modules/detect/tools/gcc/has_flag.lua
new file mode 100644
index 000000000..a0624dc4a
--- /dev/null
+++ b/xmake/modules/detect/tools/gcc/has_flag.lua
@@ -0,0 +1,114 @@
+--!The Make-like Build Utility based on Lua
+--
+-- Licensed to the Apache Software Foundation (ASF) under one
+-- or more contributor license agreements.  See the NOTICE file
+-- distributed with this work for additional information
+-- regarding copyright ownership.  The ASF licenses this file
+-- to you under the Apache License, Version 2.0 (the
+-- "License"); you may not use this file except in compliance
+-- with the License.  You may obtain a copy of the License at
+--
+--     http://www.apache.org/licenses/LICENSE-2.0
+--
+-- Unless required by applicable law or agreed to in writing, software
+-- distributed under the License is distributed on an "AS IS" BASIS,
+-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+-- See the License for the specific language governing permissions and
+-- limitations under the License.
+-- 
+-- Copyright (C) 2015 - 2017, TBOOX Open Source Group.
+--
+-- @author      ruki
+-- @file        check_flag.lua
+--
+
+-- imports
+import("lib.detect.cache")
+
+-- is linker?
+function _islinker(flag, opt)
+  
+    -- the flag is "-Wl,<arg>" or "-Xlinker <arg>"?
+    if flag:startswith("-Wl,") or flag:startswith("-Xlinker ") then
+        return true
+    end
+
+    -- the tool kind is ld or sh?
+    local toolkind = opt.toolkind or ""
+    return toolkind == "ld" or toolkind == "sh" or toolkind:endswith("-ld") or toolkind:endswith("-sh")
+end
+
+-- attempt to check it from the argument list of gcc
+function _check_from_arglist(flag, opt, islinker)
+
+    -- make cache key
+    local key = "detect.tools.gcc.has_flag"
+
+    -- make flags key
+    local flagskey = opt.program .. "_" .. (opt.programver or "")
+
+    -- load cache
+    local cacheinfo  = cache.load(key)
+
+    -- get all flags from argument list
+    local flags = cacheinfo[flagskey]
+    if not flags then
+
+        -- get argument list
+        flags = {}
+        local arglist = os.iorunv(opt.program, {"--help"})
+        if arglist then
+            for arg in arglist:gmatch("%s+(%-[%-%a]+)%s+") do
+                flags[arg] = true
+            end
+        end
+
+        -- save cache
+        cacheinfo[flagskey] = flags
+        cache.save(key, cacheinfo)
+    end
+
+    -- ok?
+    return flags[flag]
+end
+
+-- try running to check flag
+function _check_try_running(flag, opt, islinker)
+
+    -- check flag for linker
+    if islinker then
+
+        -- make an stub source file
+        local sourcefile = path.join(os.tmpdir(), "detect", "gcc_has_flag.c")
+        if not os.isfile(sourcefile) then
+            io.writefile(sourcefile, "int main(int argc, char** argv)\n{return 0;}")
+        end
+
+        -- check it
+        return try { function () os.runv(opt.program, {flag, "-o", os.nuldev(), sourcefile}); return true end }
+    end
+    
+    -- check flag for compiler
+    return try { function () os.runv(opt.program, {flag, "-S", "-o", os.nuldev(), "-xc", os.nuldev()}); return true end }
+end
+
+-- has_flag(flag)?
+-- 
+-- @param opt   the argument options, .e.g {toolname = "", program = "", programver = "", toolkind = "[cc|cxx|ld|ar|sh|gc|rc|dc|mm|mxx]"}
+--
+-- @return      true or false
+--
+function main(flag, opt)
+
+    -- is linker?
+    local islinker = _islinker(flag, opt)
+
+    -- attempt to check it from the argument list of gcc
+    if _check_from_arglist(flag, opt, islinker) then
+        return true
+    end
+
+    -- try running to check it
+    return _check_try_running(flag, opt, islinker)
+end
+
diff --git a/xmake/modules/lib/detect/has_flags.lua b/xmake/modules/lib/detect/has_flags.lua
index 43abc5a1e..bbd972c0d 100644
--- a/xmake/modules/lib/detect/has_flags.lua
+++ b/xmake/modules/lib/detect/has_flags.lua
@@ -25,36 +25,6 @@
 -- imports
 import("lib.detect.find_tool")
 
--- get flag name
---
--- .e.g 
---
--- "-fsanitize=address":                    fsanitize_address
--- "--coverage":                            coverage
--- "-nodefaultlib:\"msvcrt.lib\"":          nodefaultlib
--- "/TP":                                   tp
--- "fomit-frame-pointer":                   fomit_frame_pointer
--- "-Wno-error=deprecated-declarations":    wno_error_deprecated_declarations
--- "-pagezero_size 10000"                   pagezero_size
---
-function _get_flagname(flag)
-
-    -- get the first name by ' '
-    local name = flag:lower():split('%s+')[1]
-
-    -- split ':'
-    name = name:split(':')[1]
-
-    -- translate '-', '/' and '=' to '_'
-    name = name:gsub("[%-/=]", "_")
-
-    -- remove prefix '_*' .. xxxx
-    name = name:gsub("^_+", "")
-
-    -- ok?
-    return name
-end
-
 -- has this flag?
 function _has_flag(name, flag, opt)
 
@@ -66,12 +36,12 @@ function _has_flag(name, flag, opt)
     end
 
     -- init tool
-    opt.tool = tool
-
-    -- TODO tool.check, semver(tool.version)
+    opt.toolname   = tool.name
+    opt.program    = tool.program
+    opt.programver = tool.version
 
     -- init cache and key
-    local key     = tool.program .. "_" .. (tool.version or "") .. "_" .. flag
+    local key     = tool.program .. "_" .. (tool.version or "") .. "_" .. (opt.toolkind or "") .. "_" .. flag
     local results = _g._RESULTS or {}
     
     -- get result from the cache first
@@ -80,21 +50,13 @@ function _has_flag(name, flag, opt)
         return result
     end
 
-    -- get flag name
-    local flagname = _get_flagname(flag)
-    print(flagname)
-
-    -- "detect.tools.find_xxx" exists?
-    local result = nil
-    if os.isfile(path.join(os.programdir(), "modules", "detect", "flags", "has_ " .. flagname .. ".lua")) then
-        local hasflag = import("detect.flags.has_" .. flagname)
+    -- detect.tools.xxx.has_flag(flag, opt)?
+    if os.isfile(path.join(os.programdir(), "modules", "detect", "tools", tool.name, "has_flag.lua")) then
+        local hasflag = import("detect.tools." .. tool.name .. ".has_flag")
         if hasflag then
             result = hasflag(flag, opt)
         end
-    end
-
-    -- attempt to check it directly
-    if result == nil then
+    else
         result = try { function () os.runv(tool.program, {flag}); return true end }
     end
 
@@ -110,13 +72,14 @@ end
 --
 -- @param name      the tool name
 -- @param flags     the flags
--- @param opt       the argument options, .e.g {program = ""}
+-- @param opt       the argument options, .e.g {program = "", toolkind = "[cc|cxx|ld|ar|sh|gc|rc|dc|mm|mxx]"}
 --
 -- @return          true or false
 --
 -- @code
 -- local ok = has_flags("clang", "-g")
 -- local ok = has_flags("clang", {"-g", "-O0"}, {program = "xcrun -sdk macosx clang"})
+-- local ok = has_flags("clang", "-g", {toolkind = "cxx"})
 -- @endcode
 --
 function main(name, flags, opt)