[PATCH] D149549: [compiler-rt][interception][win] Don't crash on unknown instructions

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

@alvinhochun managed to pinpoint and guess the differing issue.

With WinSDK/UCRT 10.0.19041.0, linking fails, since references to `__stdio_common_vfprintf` pulls in `output.obj` from `libucrt.lib`, and this object file pulls in `wcstol` from `strtox.obj`, which then later triggers the duplicate definition.

With `libucrt.lib` from WinSDK 10.0.22621.0, this no longer happens. But the test still failed, fulfilling the XFAIL, with the following error:

  AddressSanitizer: CHECK failed: asan_rtl.cpp:387 "((!asan_init_is_running && "ASan init calls itself!")) != (0)" (0x0, 0x0) (tid=16328)
      <empty stack>

After this patch, this no longer triggers errors, and the test runs successfully. So the original XFAIL condition is dependent on WinSDK version, but there was a second error which made the XFAIL still seem to behave correctly, until after this change.

I can't comment on the rest of whether it's expected that this error condition that previously aborted execution, now no longer triggers a failure.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149549/new/

https://reviews.llvm.org/D149549