re-enable windows test for 0.13/LLVM 18, update README

README.md

@@ -19,27 +19,24 @@ in x86_linux, and will vary based on services used. Tested targets:
 * x86_64-linux
 * riscv64-linux\*
 * aarch64-linux
-* x86_64-windows\*\*
+* x86_64-windows
 * arm-linux
 * aarch64-macos
 * x86_64-macos
 
 Tested targets are built, but not continuously tested, by CI.
 
-\* On Zig 0.12, riscv64-linux tests take a significant time to compile (each aws.zig test takes approximately 1min, 45 seconds to compile on Intel i9 10th gen)
-
-\*\* On Zig 0.12, x86_64-windows tests have one test skipped as LLVM consumes all available RAM on the system
+\* On Zig 0.12/0.13, riscv64-linux disabled due to [LLLM's O(N^2) codegen](https://github.com/ziglang/zig/issues/18872)
 
 
 Zig-Develop Branch
 ------------------
 
 This branch is intended for use with the in-development version of Zig. This
-starts with 0.12.0-dev.3180+83e578a18. I will try to keep this branch up to date
-with latest, but with a special eye towards aligning with [Mach Engine's Nominated
-Zig Versions](https://machengine.org/about/nominated-zig/). As nightly zig versions
-disappear off the downloads page (and back end server), we can use the mirroring
-that the Mach Engine participates in to pull these versions.
+starts with 0.12.0-dev.3180+83e578a18. This is aligned with [Mach Engine's Nominated
+Zig Versions](https://machengine.org/about/nominated-zig/). Nightly zig versions
+are difficult to keep up with and there is no special effort made there, build
+status is FYI (and used as a canary for nominated zig versions).
 
 Building
 --------
@@ -52,7 +49,7 @@ project with the generated code. Testing can be done with `zig test`.
 Using
 -----
 
-This is designed for use with the Zig 0.11 package manager, and exposes a module
+This is designed for use with the Zig package manager, and exposes a module
 called "aws". Set up `build.zig.zon` and add the dependency/module to your project
 as normal and the package manager should do its thing. A full example can be found
 in [/example](example/README.md).
@@ -90,7 +87,7 @@ Services without TLS 1.3 support
 
 All AWS services should support TLS 1.3 at this point, but there are many regions
 and several partitions, and not all of them have been tested, so your mileage
-may vary.
+may vary. If something doesn't work, please submit an issue to let others know.
 
 Dependency tree
 ---------------
@@ -100,7 +97,6 @@ No dependencies:
   * aws_http_base: contains basic structures for http requests/results
   * case: provides functions to change casing
   * date: provides limited date manipulation functions
-  * http_client_17015_issue: zig 0.11 http client, with changes
   * json: custom version of earlier stdlib json parser
   * xml: custom xml parser library
   * url: custom url encoding
@@ -109,7 +105,6 @@ aws_credentials: Allows credential handling
   aws_authentication
 
 aws_http:
-  http_client_17015_issue
   aws_http_base
   aws_signing
 

build.zig

@@ -14,8 +14,10 @@ const test_targets = [_]std.zig.CrossTarget{
         .cpu_arch = .aarch64,
         .os_tag = .linux,
     },
-    // // The test executable just spins forever in LLVM using nominated zig 0.12 March 2024
-    // // This is likely a LLVM problem unlikely to be fixed in zig 0.12
+    // The test executable linking process just spins forever in LLVM using nominated zig 0.13 May 2024
+    // This is likely a LLVM problem unlikely to be fixed in zig 0.13
+    // Potentially this issue: https://github.com/llvm/llvm-project/issues/81440
+    // Zig tracker: https://github.com/ziglang/zig/issues/18872
     // .{
     //     .cpu_arch = .riscv64,
     //     .os_tag = .linux,

src/aws.zig

@@ -1904,7 +1904,6 @@ test "ec2_query_no_input: EC2 describe regions" {
 // riscv64-linux also seems to have another problem with LLVM basically infinitely
 // doing something. My guess is the @embedFile is freaking out LLVM
 test "ec2_query_with_input: EC2 describe instances" {
-    if (builtin.cpu.arch == .x86_64 and builtin.os.tag == .windows) return error.SkipZigTest;
     if (builtin.cpu.arch == .riscv64 and builtin.os.tag == .linux) return error.SkipZigTest;
     const allocator = std.testing.allocator;
     var test_harness = TestSetup.init(.{