Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM with a couple of minors
| llvm/test/CodeGen/X86/shuffle-blendw.ll | ||
|---|---|---|
| 4 | You can probably reduce the number of check lines by adding common prefixes: ; RUN: llc < %s -mtriple=i686-unknown -mattr=+sse4.1 | FileCheck %s --check-prefixes=SSE41,X86-SSE41 ; RUN: llc < %s -mtriple=x86_64-unknown -mattr=+sse4.1 | FileCheck %s --check-prefixes=SSE41,X64-SSE41 ... | |
| 10 | Add targets with AVX512 but not AVX512BW test coverage? | |
| llvm/test/CodeGen/X86/shuffle-blendw.ll | ||
|---|---|---|
| 10 | Think some knl targets have avx512f w.o bw | |
Comment Actions
@goldstein.w.n Why is this still dependent on D143787? It looks like its ready to be committed now
You can probably reduce the number of check lines by adding common prefixes: