Release 0.4.33

Fixes #42

Fix build on iOS simulator

pthread_jit_write_protect_np() is available but unusable on
iOS simulator:

../orc/orccompiler.c:460:3: error: 'pthread_jit_write_protect_np' is unavailable: not available on iOS

Fixes #43

meson: Enable only SSE and MMX backends for Windows

Other backends would not work or do not make sense

ci: move deploy stage behind build stage

orcarm.c: Implement orc_arm_flush_cache on Windows

Use the Windows API FlushInstructionCache() to flush the CPU cache on ARM64
Windows.  As a consequence, include windows.h with WIN32_LEAN_AND_MEAN defined.

orccpu-arm.c: Assume ARMv8 on Windows ARM64

ARM64 Windows are supported on ARMv8 CPUs only, so just assume that we have
the NEON and EDSP ARM instructions.

orccpu-arm.c: Include headers conditionally

unistd.h and sys/time.h may not be universally available, so only include them
if they were found at configure time.

meson: don't depend on pthreads when compiling for windows

On UNIX toolchains cross compiling for Windows, winpthread will be detected and
added as a dependency even though it's not used.

meson: export ORC_STATIC_COMPILATION in the pkgconfig file

If the library is compiled statically the define also needs to be set in the
orc-0.4.pc file so that users of the library (in a UNIX toolchain cross
compiling to Windows) will not import the functions as DLL imports.

orc: fix ORC_RESTRICT definition for MSVC

Fixes #40

meson: bump req to >= 0.55 and use meson.can_run_host_binaries()

.. instead of the deprecated meson.has_exe_wrapper()

ci: build docs and deploy to gitlab pages

https://gitlab.freedesktop.org/gstreamer/orc/-/issues/27

neon: Fix discrepancy when using loadupdb in 32-bit

Fix shifted outputs when output array is 8-byte aligned but not 16-byte aligned and loop shift is 1.

Fixes #32

Signed-off-by: Gaetan Bahl <gaetan.bahl@nxp.com>

neon: Fix testsuite not passing on arm CPUs

Set the FPCR.FZ bit before running tests using ARM NEON,
in order to make tests pass for most opcodes.

Add a way to check for expected failures in the test suite,
since ARM NEON does not comply to IEEE754.

Errors are expected when using divf (resp. sqrtf) on large
(resp. small) numbers.

Fixes #33, #20

Signed-off-by: Gaetan Bahl <gaetan.bahl@nxp.com>

neon: Fix sqrtf only processing two inputs out of four

This solves an issue where two out of four inputs are not processed
by passing the correct value of vec shift.

Fixes #33, #20.

Signed-off-by: Gaetan Bahl <gaetan.bahl@nxp.com>

neon: Fix divf only processing two inputs out of four

This solves an issue where two out of four input values are not processed
by passing the correct value of vec shift.

Fixes #33, #20.

Signed-off-by: Gaetan Bahl <gaetan.bahl@nxp.com>

neon: Fix out-of-shift errors when compiling certain float/double opcodes using neon

Setting the correct shift values solve the "out-of-shift" errors
and allow the following operators to successfully compile:
addf, subf, mulf, maxf, minf, cmpeqf, convfl, convlf, addd, subd, muld, divd

Fixes #33, #20, #2.

Signed-off-by: Gaetan Bahl <gaetan.bahl@nxp.com>

orccodemem: Fix use-after-free in error paths

In file included from gstreamer/subprojects/orc/orc/orc.h:7,
                 from gstreamer/subprojects/orc/orc/orcprogram.h:5,
                 from gstreamer/subprojects/orc/orc/orccodemem.c:30:
gstreamer/subprojects/orc/orc/orccodemem.c: In function ‘orc_code_region_allocate_codemem_dual_map.constprop’:
gstreamer/subprojects/orc/orc/orcdebug.h:138:3: warning: pointer ‘filename’ may be used after ‘free’ [-Wuse-after-free]
  138 |   orc_debug_print((level), __FILE__, ORC_FUNCTION, __LINE__, __VA_ARGS__); \
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstreamer/subprojects/orc/orc/orcdebug.h:92:26: note: in expansion of macro ‘ORC_DEBUG_PRINT’
   92 | #define ORC_WARNING(...) ORC_DEBUG_PRINT(ORC_DEBUG_WARNING, __VA_ARGS__)
      |                          ^~~~~~~~~~~~~~~
gstreamer/subprojects/orc/orc/orccodemem.c:252:5: note: in expansion of macro ‘ORC_WARNING’
  252 |     ORC_WARNING ("failed to create write map '%s'. err=%i", filename, errno);
      |     ^~~~~~~~~~~
gstreamer/subprojects/orc/orc/orccodemem.c:234:3: note: call to ‘free’ here
  234 |   free (filename);
      |   ^~~~~~~~~~~~~~~
gstreamer/subprojects/orc/orc/orcdebug.h:138:3: warning: pointer ‘filename’ may be used after ‘free’ [-Wuse-after-free]
  138 |   orc_debug_print((level), __FILE__, ORC_FUNCTION, __LINE__, __VA_ARGS__); \
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gstreamer/subprojects/orc/orc/orcdebug.h:92:26: note: in expansion of macro ‘ORC_DEBUG_PRINT’
   92 | #define ORC_WARNING(...) ORC_DEBUG_PRINT(ORC_DEBUG_WARNING, __VA_ARGS__)
      |                          ^~~~~~~~~~~~~~~
gstreamer/subprojects/orc/orc/orccodemem.c:245:5: note: in expansion of macro ‘ORC_WARNING’
  245 |     ORC_WARNING("failed to create exec map '%s'. err=%i", filename, errno);
      |     ^~~~~~~~~~~
gstreamer/subprojects/orc/orc/orccodemem.c:234:3: note: call to ‘free’ here
  234 |   free (filename);
      |   ^~~~~~~~~~~~~~~

Fixes: bb5fcb31 ("orccodemem: Report errno during failures to create mmap codemap.")

ci: Add ARM64 and ARM Windows UWP jobs

Cross and native files are modified versions of the files used in the
gstreamer CI since we use the same Docker image.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/64>

ci: Update Windows image and macos/ios tags

The latest image contains VS 2019, and was built in
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1570

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/63>

neon: Fix unsigned only implementation of loadoffb, loadoffw and loadoffl

This has direct impact on bayer2rgb performance. Tested on i.MX8mm aarch64 -> Speedboost of ~17%.
Reason:
The line loadoffw t, s, -1 results in orc silent compile error
Pipeline:
gst-launch-1.0 -v videotestsrc ! video/x-bayer,width=1920,height=1080 ! bayer2rgb ! fpsdisplaysink video-sink=fakesink sync=0
Average performance with fix: 25.21fps
Average performance without fix: 21.60fps

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/62>

meson: Disable ARM64 support for Windows

Disable Windows ARM64 support/build for now, because it doesn't work.

Fixes: https://gitlab.freedesktop.org/gstreamer/orc/-/issues/36
Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/61>

executor: Fix orc_executor_set_program() to save OrcCode

orc_executor_set_program() was missed when it was added to
orc_executor_new().

meson: Set recursive for extract_all_objects()

Fixes the following warning:

WARNING: extract_all_objects called without setting recursive
keyword argument. Meson currently defaults to
non-recursive to maintain backward compatibility but
the default will be changed in the future.
orc-test\meson.build:16:0: ERROR: Fatal warnings enabled, aborting

orc: Add support for MacOS Hardened runtime.

orccodemem: Report errno during failures to create mmap codemap.

orc/orccpu-powerpc.c: fix build with kernel < 4.11

Build with powerpc and kernel < 4.11 is broken since version 0.4.30 and
https://gitlab.freedesktop.org/gstreamer/orc/-/commit/a999325abea6a5549d60d99ddeb0271d2aa00235:

FAILED: orc/liborc-0.4.so.0.32.0.p/orccpu-powerpc.c.o
/home/giuliobenetti/autobuild/run/instance-3/output-1/host/bin/powerpc-linux-gcc -Iorc/liborc-0.4.so.0.32.0.p -Iorc -I../orc -I. -I.. -fdiagnostics-color=always -pipe -Wall -Winvalid-pch -std=gnu99 -O3 -DHAVE_CONFIG_H -fvisibility=hidden -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -O2 -fPIC -pthread -DORC_ENABLE_UNSTABLE_API -D_GNU_SOURCE -DBUILDING_ORC -MD -MQ orc/liborc-0.4.so.0.32.0.p/orccpu-powerpc.c.o -MF orc/liborc-0.4.so.0.32.0.p/orccpu-powerpc.c.o.d -o orc/liborc-0.4.so.0.32.0.p/orccpu-powerpc.c.o -c ../orc/orccpu-powerpc.c
../orc/orccpu-powerpc.c: In function 'orc_check_powerpc_proc_auxv':
../orc/orccpu-powerpc.c:164:21: error: 'AT_L1D_CACHESIZE' undeclared (first use in this function); did you mean 'AT_DCACHEBSIZE'?
  164 |       if (buf[i] == AT_L1D_CACHESIZE) {
      |                     ^~~~~~~~~~~~~~~~
      |                     AT_DCACHEBSIZE
../orc/orccpu-powerpc.c:164:21: note: each undeclared identifier is reported only once for each function it appears in
../orc/orccpu-powerpc.c:168:21: error: 'AT_L2_CACHESIZE' undeclared (first use in this function); did you mean 'AT_ICACHEBSIZE'?
  168 |       if (buf[i] == AT_L2_CACHESIZE) {
      |                     ^~~~~~~~~~~~~~~
      |                     AT_ICACHEBSIZE
../orc/orccpu-powerpc.c:172:21: error: 'AT_L3_CACHESIZE' undeclared (first use in this function); did you mean 'AT_ICACHEBSIZE'?
  172 |       if (buf[i] == AT_L3_CACHESIZE) {
      |                     ^~~~~~~~~~~~~~~
      |                     AT_ICACHEBSIZE

Indeed, AT_{L1D,L2,L3}_CACHESIZE is only defined since kernel 4.11 and
https://github.com/torvalds/linux/commit/98a5f361b8625c6f4841d6ba013bbf0e80d08147

Fixes:
 - http://autobuild.buildroot.org/results/0821e96cba3e455edd47b87485501d892fc7ac6a

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/56>

meson: don't set gnu99 as default c_std

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/55>

aarch32: Implement loadupdb instruction

Fill in aarch32 opcodes for loadupdb instruction, which is used
by various color space conversion programs. There is likely still
some space for optimization.

aarch64: Implement loadupdb instruction

Fill in aarch64 opcodes for loadupdb instruction, which is used
by various color space conversion programs. This is thus far only
available on aarch64, but arm32 port should be easy.

aarch64: Implement sqrtf instruction

Fill in aarch64 opcodes for sqrtf instruction.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement divf instruction

Fill in aarch64 opcodes for divf instruction.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement double-precision floating-point arithmetic instructions

Fill in aarch64 opcodes for double-precision floating point
arithmetic instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement flags2d

Implement support for .flags 2d by adding code for handling the loop
counter. The implementation is very similar to aarch32.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement const64 loadiq

Add support for loading 64bit constants on aarch64 by emitting a
sequence of pc-relative load, branch past the literal pool and a
constant as a literal pool entry.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement accumulator store

Add code to store the accumulator register.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement floating-point arithmetic instructions

Fill in aarch64 opcodes for floating point arithmetic instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement andn instruction

Fill in aarch64 opcode for andn instruction.
This is bic instruction with reversed operands.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement splatw3q instruction

This uses TRN2 instruction twice on the same data, first to expand
top half-word of SRC register into TMP register and then to expand
top world of TMP register to DST register. The following scheme is
implemented: src[ABCDEFGH] -> tmp[ABABEFEF] -> dst[ABABABAB] .

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement swapX instructions

Fill in aarch64 opcodes for swapX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement div255w instruction

Fill in aarch64 opcodes for div255w instruction.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement mulX instructions

Fill in aarch64 opcodes for mulX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement cmpX instructions

Fill in aarch64 opcodes for cmpX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement avgX instructions

Fill in aarch64 opcodes for avgX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement splitX/splatX instructions

Fill in aarch64 opcodes for splitX/splatX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement signX instructions

Fill in aarch64 opcodes for signX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement vminX/vmaxX instructions

Fill in aarch64 opcodes for vminX/vmaxX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement accX instructions

Fill in aarch64 opcodes for accX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement loadiX instructions

Fill in aarch64 opcodes for loadiX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement subX instructions

Fill in aarch64 opcodes for subX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement andX instructions

Fill in aarch64 opcodes for andX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement absX instructions

Fill in aarch64 opcodes for absX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement xorX instructions

Fill in aarch64 opcodes for xorX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement copyX/orX instructions

Fill in aarch64 opcodes for copyX/orX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement mergeX instructions

Fill in aarch64 opcodes for mergeX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Clean up mergeX/splatX instructions

Factor out common code from the mergeX/splatX instructions so it
is easier to add the aarch64 counterpart.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement loadX instructions

Fill in aarch64 opcodes for loadX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement shift instructions

Fill in aarch64 opcodes for shift instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement mov instructions

Fill in aarch64 opcodes for mov instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement mulhX instructions

Fill in aarch64 opcodes for mulhX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement select{0,1}X instructions

Fill in aarch64 opcodes for select{0,1}X instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement convX instructions

Fill in aarch64 opcodes for convX instructions.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Implement unary instruction emit

Fill in missing code to emit unary instructions for aarch64 neon.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Fix MSVC warnings

This fixes various warnings reported by the MSVC in CI.
No functional change.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Repair storeX instructions

The storeX instructions are a mirror image of loadX instructions, however
the current implementation is a copy and the storeX arguments are swapped.
This does lead to incorrect store and either to segfault or invalid data.
Fix this by correcting the storeX instruction arguments.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Repair emit for imm 1

The logical immediate encoding of 1 is 0, so encode_logical_imm()
would return 0 in that case. However, 0 is treated as a failure.
Adjust encode_logical_imm() such that it return a negative return
value in case of failure, 0 in case of success and the logical
encoding of immediate value is returned via parameter.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Repair 8bit load/store opcode

The opcode for 8bit load/store is 0 instead of 1, fix this.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: Use 64bit operations on 64bit pointers

This fixes loading values from arrays, where the array pointers
are 64bit, but the operation incrementing the pointer was 32bit
and thus the top 32bits of the pointer were set to zero. Fix it
by using 64bit operations on the pointer.

Signed-off-by: Marek Vasut <marex@denx.de>

aarch64: orcprogram-neon porting to aarch64

This PR ports orcprogram-neon.c for AArch64 support.
It makes the orc compiler generate aarch64 assembly codes.

Currently, example1 succesfully runs on aarch64 (tested on an Android device).

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

aarch64: implement emits for some vector instructions and ORC ops (add)

This commit implements vector load/store/add instructions and ORC 'add' ops.
They reuse existing macros like 'BINARY', and aarch64-specific arguments are added.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

aarch64: implement emits for general instructions

This commit implement emits for general instructions.
Also, it defines a bunch of macros that can be used in orcprogram-neon.c

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

aarch64: make some setups for aarch64 support

This commit adds the setup codes for aarch64 support including meson build and aarch64 selection.

Signed-off-by: Dongju Chae <dongju.chae@samsung.com>

meson: Get rid of outdated version_compare checks

ci: Also pass --fatal-meson-warnings

To ensure that warnings don't sneak in.

meson: Bump version to 0.54 to fix warnings

Release 0.4.32

Fixes #31

ci: Add an Android arm64 job run on Fedora

Uses the same image as the android examples job on the cerbero CI.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/51>

ci: Add iOS CI with a dynamic cross file

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/51>

ci: Use the latest meson syntax

Also rename macos job for consistency. Install ninja using pip on
Windows to workaround https://github.com/mesonbuild/meson/pull/7669

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/51>

ci: Use werror on macOS too

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/51>

meson: Use pkgconfig.generate

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/49>

ci: mark the msys job as manual and allowed to fail

the msys setup is not deterministic due to issues with msys installation,
so it keeps breaking every now and then.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/48>

orc: Use page size aligned memory block for Windows

VirtualProtect will change protection attribute of all memory range
which belong to the same page range.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/48>

orc: Remove trailing white spaces

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/48>

ci: add basic macOS CI

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/46>

meson: use / instead of join_paths() for readability

ci: run benchmark tests as well

meson: hook up benchmark as benchmark

benchmorc: fix file path to test orc file on windows

Fix compiler warning on msys2:
enchmorc.c:35:31: error: unknown escape sequence: '\o'

The absolute path returned by current_source_dir()
returns backward slashes, which don't work well when
used as a C string constant. join_paths() will make
it all forward-slashes.

orcc: use #pragma once in generated headers

Simplifies code and removes build path fragments from
generated file (paths may be different if generated
in a gst-build setup).

There shouldn't be any portability issues with this. Meson
has been using this in its generated config.h for years.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/45>

orc: Don't set memory protection if there's no code

If the size of the JIT code is 0, there's no code and the *mem is
uninitialized. This can happen when orcc.exe is used to generate
backup C code.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/44>

orc: Print hex value directly instead of a constant

This is actually more useful because the constants are all bitfields
and it's fairly straightforward to look it up:

https://docs.microsoft.com/en-us/windows/win32/memory/memory-protection-constants

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/44>

orc: Fix indentation

Only whitespace changes.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/44>

ci: Attempt to fix broken Windows CI

https://gitlab.freedesktop.org/nirbheek/orc/-/pipelines/169274

Update to latest image and use the gstreamer runner tag. Matches the
gst-ci template.

Also try to make MSYS2 CI more resilient by following:
https://github.com/msys2/setup-msys2/blob/master/main.js#L98

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/42>

orc: Fix some warnings on MSVC and enable --werror

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/42>

orccodemem: Add support for Universal Windows Platform apps

VirtualAlloc is VirtualAllocFromApp when targeting UWP, and you can
only allocate executable pages if you have the codeGeneration
capability set in the app manifest. Check for that capability in
_orc_compiler_init() and switch to backup code or emulation if it
isn't available instead of crashing when VirtualAllocFromApp
returns NULL.

Also you cannot allocate pages that are both READWRITE and EXECUTE, so
we allocate as read-write first, then set the memory as execute-only
after the code has been compiled and copied over.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/42>

orc: Implement a windows helper for getenv()

On Windows, getenv() is deprecated and does not work in all cases. On
the Universal Windows Platform (UWP) it always returns NULL. Add
a wrapper orc_getenv() that calls GetEnvironmentVariable on Windows.

Also change semantics to always make a copy before returning.

Part-of: <https://gitlab.freedesktop.org/gstreamer/orc/-/merge_requests/42>

meson: improve summary() printout

Pass bool_yn kwarg to summary() to make it
print boolean arguments as nice coloured YES/NO
instead of true/false.

We can also pass multiple arguments like a bool
and a disabled_reason string. In meson 0.54 these
can be printed on one line if we set the line_sep
kwarg. In meson 0.53 these will always be printed
on two lines (and it will warn about the line_sep
arg), so only pass two args if docs are disabled
and otherwise just pass one arg, so we don't end
up with an ugly empty line with meson 0.53.

meson: add summary() if Meson version is new enough

ci: update windows runner tag and image