[PATCH] D78609: [llvm] [X86] Add "REQUIRES asserts" to test

Aart Bik via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 21 23:12:36 PDT 2020


aartbik added a comment.

Hmm, given that some builds now break on using "extract_subvector" instead of "vselect",
I am actually really start to like the suggestion to just test the assembly and rely on
not hitting asserts in builds that support these, but not let the test rely on debug output.

CHECK: v4f64 = vselect

<stdin>:8:40: note: scanning from here
Initial selection DAG: %bb.0 'bug45563:'

<stdin>:41:7: note: possible intended match here
 t54: v2f64 = extract_subvector t52, Constant:i64<0


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78609





More information about the llvm-commits mailing list