[PATCH] D106780: [x86] Fix lowering to illegal type in LowerINSERT_VECTOR_ELT

Mon Jul 26 02:11:26 PDT 2021

RKSimon added a comment.

Nice catch!

================
Comment at: llvm/test/CodeGen/X86/vector_double2_insert.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc < %s -O0 -mtriple i686-pc-win32-msvc -mcpu=skx               | FileCheck %s
+
----------------
We'd probably be better off adding i686 triple coverage to llvm/test/CodeGen/X86/avx-insertelt.ll to get much better coverage:
```
; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx  | FileCheck %s --check-prefixes=ALL,AVX
; RUN: llc < %s -mtriple=x86_64-unknown-unknown -mattr=+avx2 | FileCheck %s --check-prefixes=ALL,AVX2,X64-AVX2
; RUN: llc < %s -mtriple=i686-unknown-unknown -mattr=+avx2 | FileCheck %s --check-prefixes=ALL,AVX2,X86-AVX2
```

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

https://reviews.llvm.org/D106780