[PATCH] D121973: Recommit "[SLP] Fix lookahead operand reordering for splat loads."

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

this causes crashes:

  $ cat /tmp/a.ll
  target triple = "x86_64-unknown-linux-gnu"
  
  define internal fastcc void @0() #0 {
    %1 = load double, double* null, align 8
    %2 = fcmp ogt double 0.000000e+00, %1
    %3 = fcmp ogt double 0.000000e+00, %1
    ret void
  }
  
  attributes #0 = { "target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+sse3,+x87" }
  
  $ bin/opt -passes=slp-vectorizer -disable-output /tmp/a.ll
  opt: ../../llvm/lib/Target/X86/X86TargetTransformInfo.cpp:1563: llvm::InstructionCost llvm::X86TTIImpl::getShuffleCost(TTI::ShuffleKind, llvm::VectorType *, ArrayRef<int>, int, llvm::VectorType *, ArrayRef<llvm::Value *>): Assertion `isLegalBroadcastLoad( BaseTp->getElementType(), cast<FixedVectorType>(BaseTp)->getNumElements()) && "Table entry missing from isLegalBroadcastLoad()"' failed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121973