[PATCH] D105632: [LV] Use lookThroughAnd with logical reductions

[Kerry McLaughlin via Phabricator via llvm-commits]

kmclaughlin added a comment.

In D105632#2893018 <https://reviews.llvm.org/D105632#2893018>, @morehouse wrote:

> This patch appears to have broken the sanitizer-windows bot:  https://lab.llvm.org/buildbot/#/builders/127/builds/14307
>
>   ******************** TEST 'AddressSanitizer-x86_64-windows :: TestCases/Windows/recalloc_sanity.cpp' FAILED ********************
>   Exit Code: 1318
>   # command output:
>      Creating library C:\b\slave\sanitizer-windows\build\stage1\projects\compiler-rt\test\asan\X86_64WindowsConfig\TestCases\Windows\Output\recalloc_sanity.cpp.tmp.lib and object C:\b\slave\sanitizer-windows\build\stage1\projects\compiler-rt\test\asan\X86_64WindowsConfig\TestCases\Windows\Output\recalloc_sanity.cpp.tmp.exp
>   LINK : fatal error LNK1318: Unexpected PDB error; FORMAT (11) ''
>   # command stderr:
>   clang-cl: error: linker command failed with exit code 1318 (use -v to see invocation)
>   error: command failed with exit status: 1318
>
> Could you please take a look?

Hi @morehouse,
I have reverted this patch for now in case this is causing the failures, although I believe the sanitizer-windows bot is still failing with the reverted changes: https://lab.llvm.org/buildbot/#/builders/127/builds/14324
Looking at the LINK errors here <https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-error-lnk1318?view=msvc-160>, I think this can happen due to stale files which weren't deleted from a previous run and it's possible this may not be related to my patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105632