[all-commits] [llvm/llvm-project] c1899c: [X86][AVX] Add PR48908 shuffle test case

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Wed Feb 3 11:28:06 PST 2021


  Branch: refs/heads/release/12.x
  Home:   https://github.com/llvm/llvm-project
  Commit: c1899cd5102dbdacd006fdb33db075319ccc933f
      https://github.com/llvm/llvm-project/commit/c1899cd5102dbdacd006fdb33db075319ccc933f
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M llvm/test/CodeGen/X86/vector-shuffle-combining-avx.ll

  Log Message:
  -----------
  [X86][AVX] Add PR48908 shuffle test case

(cherry picked from commit da8845fc3d3bb0b0e133f020931440511fa72723)


  Commit: 52a70a07e93c322ad137bce1a1ff2f1c9fdf6050
      https://github.com/llvm/llvm-project/commit/52a70a07e93c322ad137bce1a1ff2f1c9fdf6050
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M llvm/lib/Target/X86/X86ISelLowering.cpp
    M llvm/test/CodeGen/X86/vector-shuffle-combining-avx.ll

  Log Message:
  -----------
  [X86][AVX] canonicalizeLaneShuffleWithRepeatedOps - don't merge VPERMILPD ops with different low/high masks.

Unlike VPERMILPS, VPERMILPD can have non-repeating masks in each 128-bit subvector, we weren't accounting for this when folding vperm2f128(vpermilpd(x,c),vpermilpd(y,c)) -> vpermilpd(vperm2f128(x,y),c).

I'm intending to add support for this but wanted to get a minimal fix in first for merging into 12.xx.

Fixes PR48908

(cherry picked from commit 6663330bc8c84a75ea092272297b557bfc310380)


Compare: https://github.com/llvm/llvm-project/compare/2a57ea296a47...52a70a07e93c


More information about the All-commits mailing list