from marauder:
Fixing transform plane as some descrepencies between the simd function and the c math caused issues with precision.
Use c math for inverse functions until a proper simd implementation can be found.
In Player::updatePos(), when processing physics collision results, the code was incorrectly using col.object instead of colCheck.object when checking if the collision object is a player.
This caused:
- First iteration: col.object is uninitialized (zeroed), leading to null pointer access
- Subsequent iterations: col.object contains the previous iteration's value, causing incorrect type checks
The fix changes col.object to colCheck.object to properly check the current collision object.
the opened ddsfs file for pulling metadata was leaking. The FileStream function close does not delete the pointer to the original file only FileObject does this. It might be worthwhile instead to switch to using fileObject anywhere we use a filestream that way we can be at least somewhat safe in using the object and once it goes out of scope the raw pointer will be freed
wrap safety around normal checks, this was done on the scalar math may as well do it here just in case.
Ad the impl.inl files to libraries so they can actually be found.
removed some x86 intrinsic functions that were in the mat44_impl file
reinstated some mMath_C functions and mMathFn ptrs trying to diagnose an issue.
Had to come up with a different way to initialize the scalar table if the isa tables are not initialized yet. Mac did not like the static initialization.
Had to change neon over to using explicit masks for shifting, cross product was failing during bakes and matrix calculations
most matrix functions are converted over, no benefit to converting over the project/ortho because they would be scalar anyway but may need to move them regardless.
working for both neon32 and neon64
Update math_backend.cpp
further sse simd additions
avx2 float3 added
added normalize_magnitude
added divide fast to float3 may copy to float4
move static spheremesh to drawSphere (initialize on first use) so platform has a chance to load the math backend
all float3 and float4 functions and isas
completed all options of float3 and float4 functions in isas and math_c
neon still to be done but that will be on mac.
Update math_backend.cpp
mac isa neon update
added float3
restructured the classes to look more like the final version of the x86 classes
linux required changes
Update build-macos-clang.yml
Update build-macos-clang.yml
Revert "Update build-macos-clang.yml"
This reverts commit 29dfc567f4.
Revert "Update build-macos-clang.yml"
This reverts commit 2abad2b4ca.
Update CMakeLists.txt
fix macs stupid build
remove god awful rolling average from frame time tracker....
use intrinsic headers instead
each isa implementation now uses a header for that isa's intrinsic functions these are then used in the impl files. This will make it easier for matrix functions when those are implemented.
fixed comment saying 256 when it should be 512 for avx512
consolidated initializers for function tables
Update neon_intrinsics.h
fixes for some neon intrinsics no idea if this is the best way to do these but they work at least
v_cross is especially messy at the moment we basically just do it as a c math function need to look into getting this done correctly
and from Marauder, hardened filename entry for con::evaluate/compileexec. if the string forwarded along is not a real filename, consider it an eval block
for context Strings, use c_str(),
for triggers, use Con::getCurrentScriptModulePath() as otherwise, the "%this =" and "%obj=" injections will be treated as within the global scope. which is not allowed.
