repo: try to reduce userdata by 16 bytes

Author: RaphaelIT7

.github/workflows/compile-luajit.yml

@@ -4,6 +4,7 @@ on:
   workflow_dispatch:
   push:
     paths:
+      - '.github/workflows/compile-luajit.yml'
       - 'libs/liblua_shared_lib_*'
       - 'libs/lua_shared_lib_*'
       - 'luajit/**'

.github/workflows/show_summary.yml

@@ -1,4 +1,4 @@
-name: Build JIT
+name: Show Summary
 
 on:
   workflow_dispatch:

source/_prebuildtools/_workflow_summary.lua

@@ -69,7 +69,7 @@ function FetchFromHolyLogs(github_repository, runNumber, host, apikey, previous_
 
 	if previous_run_number > 0 then
 		local lastHolyLogsResults = {}
-		FetchHolyLogsResults(github_repository, runNumber, function(jsonTable)
+		FetchHolyLogsResults(github_repository, previous_run_number, function(jsonTable)
 			lastHolyLogsResults = jsonTable
 		end, host, apikey)
 
@@ -200,7 +200,7 @@ function CalculateDifferences(currentResults, previousResults)
 			end
 
 			funcResults.timePerCall = funcResults.totalTime / funcResults.totalCalls
-			funcResults.diffTimePerCall = prevFuncResults and (funcResults.timePerCall / prevFuncResults.timePerCall) or 1
+			funcResults.diffTimePerCall = prevFuncResults and (prevFuncResults.timePerCall / funcResults.timePerCall) or 1
 		end
 	end
 end
@@ -219,8 +219,12 @@ Previous run: ]] .. previousRun .. "<br>")
 		table.insert(markdown, "| ------------- | ----------- | ------------- | ---------------------- |")
 
 		for funcName, funcResults in pairs(funcs) do
-			if math.abs(1 - funcResults.diffTimePerCall) > 0.20 then -- It may fluxuate between builds because of different runners/hardware but it shouldn't worsen by more than 20%
-				print("::warning title=" .. funcName .. "::Performance is worse beyond expectation")
+			local diff = funcResults.diffTimePerCall
+			local maxDiff = 0.20 -- It may fluxuate between builds because of different runners/hardware but it shouldn't worsen by more than 20%
+			if diff > (1 + maxDiff) then
+				print("::warning title=" .. funcName .. "::Performance is worse beyond expectation (" .. string.format("%.2fx", diff) .. " slower)")
+			elseif diff < (1 - maxDiff) then
+				print("::notice title=" .. funcName .. "::Performance improved significantly (" .. string.format("%.2fx", 1 / diff) .. " faster)")
 			end
 
 			table.insert(markdown, "| " .. funcName .. " | " .. funcResults.totalCalls .. " | " .. funcResults.timePerCall .. " | " .. funcResults.diffTimePerCall .. "x |")

source/detouring/README.md

@@ -0,0 +1,2 @@
+We need a custom version since the original does NOT allow one to hook multiple instances if they have different VTables.<br>
+We hit this special case with the `CLuaInterfaceProxy` inside of `lua.cpp` were we can hook both into Gmod's `ILuaInterface` VTable but **also** into our own `ILuaInterface` VTable which wasn't possible previously and it only hooked into the first VTable while for the second one it never hooked into it even though it would report hooking as successful.