mirror of
https://github.com/luau-lang/luau.git
synced 2024-11-15 06:15:44 +08:00
e6bf71871a
Some checks failed
benchmark / callgrind (map[branch:main name:luau-lang/benchmark-data], ubuntu-22.04) (push) Has been cancelled
build / ${{matrix.os.name}} (map[name:macos version:macos-latest]) (push) Has been cancelled
build / ${{matrix.os.name}} (map[name:macos-arm version:macos-14]) (push) Has been cancelled
build / ${{matrix.os.name}} (map[name:ubuntu version:ubuntu-latest]) (push) Has been cancelled
build / windows (Win32) (push) Has been cancelled
build / windows (x64) (push) Has been cancelled
build / coverage (push) Has been cancelled
build / web (push) Has been cancelled
release / ${{matrix.os.name}} (map[name:macos version:macos-latest]) (push) Has been cancelled
release / ${{matrix.os.name}} (map[name:ubuntu version:ubuntu-20.04]) (push) Has been cancelled
release / ${{matrix.os.name}} (map[name:windows version:windows-latest]) (push) Has been cancelled
release / web (push) Has been cancelled
Instead of doing the dot product related math in scalar IR, we lift the computation into a dedicated IR instruction. On x64, we can use VDPPS which was more or less tailor made for this purpose. This is better than manual scalar lowering that requires reloading components from memory; it's not always a strict improvement over the shuffle+add version (which we never had), but this can now be adjusted in the IR lowering in an optimal fashion (maybe even based on CPU vendor, although that'd create issues for offline compilation). On A64, we can either use naive adds or paired adds, as there is no dedicated vector-wide horizontal instruction until SVE. Both run at about the same performance on M2, but paired adds require fewer instructions and temporaries. I've measured this using mesh-normal-vector benchmark, changing the benchmark to just report the time of the second loop inside `calculate_normals`, testing master vs #1504 vs this PR, also increasing the grid size to 400 for more stable timings. On Zen 4 (7950X), this PR is comfortably ~8% faster vs master, while I see neutral to negative results in #1504. On M2 (base), this PR is ~28% faster vs master, while #1504 is only about ~10% faster. If I measure the second loop in `calculate_tangent_space` instead, I get: On Zen 4 (7950X), this PR is ~12% faster vs master, while #1504 is ~3% faster On M2 (base), this PR is ~24% faster vs master, while #1504 is only about ~13% faster. Note that the loops in question are not quite optimal, as they store and reload various vectors to dictionary values due to inappropriate use of locals. The underlying gains in individual functions are thus larger than the numbers above; for example, changing the `calculate_normals` loop to use a local variable to store the normalized vector (but still saving the result to dictionary value), I get a ~24% performance increase from this PR on Zen4 vs master instead of just 8% (#1504 is ~15% slower in this setup). |
||
|---|---|---|
| .. | ||
| conformance | ||
| require | ||
| AnyTypeSummary.test.cpp | ||
| AssemblyBuilderA64.test.cpp | ||
| AssemblyBuilderX64.test.cpp | ||
| AstJsonEncoder.test.cpp | ||
| AstQuery.test.cpp | ||
| AstQueryDsl.cpp | ||
| AstQueryDsl.h | ||
| AstVisitor.test.cpp | ||
| Autocomplete.test.cpp | ||
| BuiltinDefinitions.test.cpp | ||
| ClassFixture.cpp | ||
| ClassFixture.h | ||
| CodeAllocator.test.cpp | ||
| Compiler.test.cpp | ||
| Config.test.cpp | ||
| Conformance.test.cpp | ||
| ConformanceIrHooks.h | ||
| ConstraintGeneratorFixture.cpp | ||
| ConstraintGeneratorFixture.h | ||
| ConstraintSolver.test.cpp | ||
| CostModel.test.cpp | ||
| DataFlowGraph.test.cpp | ||
| DenseHash.test.cpp | ||
| DiffAsserts.cpp | ||
| DiffAsserts.h | ||
| Differ.test.cpp | ||
| EqSat.language.test.cpp | ||
| EqSat.propositional.test.cpp | ||
| EqSat.slice.test.cpp | ||
| EqSatSimplification.test.cpp | ||
| Error.test.cpp | ||
| Fixture.cpp | ||
| Fixture.h | ||
| FragmentAutocomplete.test.cpp | ||
| Frontend.test.cpp | ||
| Generalization.test.cpp | ||
| InsertionOrderedMap.test.cpp | ||
| Instantiation2.test.cpp | ||
| IostreamOptional.h | ||
| IrBuilder.test.cpp | ||
| IrCallWrapperX64.test.cpp | ||
| IrLowering.test.cpp | ||
| IrRegAllocX64.test.cpp | ||
| JsonEmitter.test.cpp | ||
| Lexer.test.cpp | ||
| Linter.test.cpp | ||
| LValue.test.cpp | ||
| main.cpp | ||
| Module.test.cpp | ||
| NonstrictMode.test.cpp | ||
| NonStrictTypeChecker.test.cpp | ||
| Normalize.test.cpp | ||
| NotNull.test.cpp | ||
| Parser.test.cpp | ||
| RegisterCallbacks.cpp | ||
| RegisterCallbacks.h | ||
| Repl.test.cpp | ||
| RequireByString.test.cpp | ||
| RequireTracer.test.cpp | ||
| RuntimeLimits.test.cpp | ||
| ScopedFlags.h | ||
| Set.test.cpp | ||
| SharedCodeAllocator.test.cpp | ||
| Simplify.test.cpp | ||
| StringUtils.test.cpp | ||
| Subtyping.test.cpp | ||
| Symbol.test.cpp | ||
| ToDot.test.cpp | ||
| TopoSort.test.cpp | ||
| ToString.test.cpp | ||
| Transpiler.test.cpp | ||
| TxnLog.test.cpp | ||
| TypeFunction.test.cpp | ||
| TypeFunction.user.test.cpp | ||
| TypeInfer.aliases.test.cpp | ||
| TypeInfer.annotations.test.cpp | ||
| TypeInfer.anyerror.test.cpp | ||
| TypeInfer.builtins.test.cpp | ||
| TypeInfer.cfa.test.cpp | ||
| TypeInfer.classes.test.cpp | ||
| TypeInfer.definitions.test.cpp | ||
| TypeInfer.functions.test.cpp | ||
| TypeInfer.generics.test.cpp | ||
| TypeInfer.intersectionTypes.test.cpp | ||
| TypeInfer.loops.test.cpp | ||
| TypeInfer.modules.test.cpp | ||
| TypeInfer.negations.test.cpp | ||
| TypeInfer.oop.test.cpp | ||
| TypeInfer.operators.test.cpp | ||
| TypeInfer.primitives.test.cpp | ||
| TypeInfer.provisional.test.cpp | ||
| TypeInfer.refinements.test.cpp | ||
| TypeInfer.singletons.test.cpp | ||
| TypeInfer.tables.test.cpp | ||
| TypeInfer.test.cpp | ||
| TypeInfer.tryUnify.test.cpp | ||
| TypeInfer.typePacks.test.cpp | ||
| TypeInfer.typestates.test.cpp | ||
| TypeInfer.unionTypes.test.cpp | ||
| TypeInfer.unknownnever.test.cpp | ||
| TypePack.test.cpp | ||
| TypePath.test.cpp | ||
| TypeVar.test.cpp | ||
| Unifier2.test.cpp | ||
| Variant.test.cpp | ||
| VecDeque.test.cpp | ||
| VisitType.test.cpp | ||