[PATCH] D115955: [SLP]Alternate vectorization for cmp instructions.

Wed Feb 2 14:26:52 PST 2022

ABataev added a comment.

In D115955#3292014 <https://reviews.llvm.org/D115955#3292014>, @srj wrote:

> Update: one of Halide's build-in tests fails reliably (for me) with these patches in place; if you pull and build Halide locally, it may make it easier for you to reproduce the failures locally. Here are the steps I think are necessary:
>
> - git clone https://github.com/halide/Halide.git /path/to/halide
> - cd /path/to/halide
> - git checkout master
> - export LLVM_CONFIG=/path/to/llvm-install/bin/llvm-config
> - export LLVM_DIR=/path/to/llvm-install/lib/cmake/llvm
> - export HL_TARGET=x86-64-linux-sse41   # Assumes you are on an x86-64 Linux machine w/ SSE4.1
> - make -j8 generator_aot_metadata_tester
>
> (wait for build & test to complete)
>
> If everything is fine, this will produce something like `Success!`.
> If broken, this will produce something like `Error: Total allocation for buffer array_outputs6_0 is 1024, which exceeds the maximum size of 2147483647 (Aborted)`

Hi, thanks, already found a bug using your previous archive, it was a misprint I overlooked, will commit fixed version tomorrow.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115955