[PATCH] D51553: [DAGCombiner][x86] add transform/hook to load a scalar directly for use in a vector binop

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 1 09:32:36 PDT 2018


spatel added inline comments.


================
Comment at: test/CodeGen/X86/load-scalar-as-vector.ll:5
 ; RUN: llc < %s -mtriple=x86_64-- -mattr=+avx        | FileCheck %s --check-prefixes=AVX,AVX1
 ; RUN: llc < %s -mtriple=x86_64-- -mattr=+avx2       | FileCheck %s --check-prefixes=AVX,AVX2
 
----------------
RKSimon wrote:
> Please can you rebase as I added avx512 tests at rL342773 and possibly add some tests with 256/512 vectors as well?
Sure - there's no difference for AVX512 with the current set of tests on this patch, but there should be after rL343491.


https://reviews.llvm.org/D51553





More information about the llvm-commits mailing list