[PATCH] D38495: [X86] Fix bug in legalize vector types - Split large loads

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 2 03:11:36 PDT 2017


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM once the tests have been split up



================
Comment at: test/CodeGen/X86/pr34653-pr34657.ll:3
+; RUN: llc %s -O0 -mtriple=x86_64-unknown-unknown -mattr=+avx512f | FileCheck %s -check-prefix=CHECKO0
+; RUN: llc %s -O2 -mtriple=x86_64-unknown-unknown -mattr=+avx512f,+avx512bw | FileCheck %s -check-prefix=CHECKO2
+
----------------
aymanmus wrote:
> RKSimon wrote:
> > Why the difference in attributes?
> To reproduce both of the bugs as they were filed.
OK - please can you just split these into 2 test files. pr34653.ll and pr34657.ll


https://reviews.llvm.org/D38495





More information about the llvm-commits mailing list