[all-commits] [llvm/llvm-project] 8590d2: [SLP] move reduction createOp functions; NFC

[RotateRight via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8590d245434dd4205c89f0a05b4c22feccb7421c
      https://github.com/llvm/llvm-project/commit/8590d245434dd4205c89f0a05b4c22feccb7421c
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2021-01-20 (Wed, 20 Jan 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

  Log Message:
  -----------
  [SLP] move reduction createOp functions; NFC

We were able to remove almost all of the state from
OperationData, so these don't make sense as members
of that class - just pass the RecurKind in as a param.


  Commit: 1c54112a5762ebab2c14a90c55f27d00bfced7f8
      https://github.com/llvm/llvm-project/commit/1c54112a5762ebab2c14a90c55f27d00bfced7f8
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2021-01-20 (Wed, 20 Jan 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

  Log Message:
  -----------
  [SLP] refactor more reduction functions; NFC

We were able to remove almost all of the state from
OperationData, so these don't make sense as members
of that class - just pass the RecurKind in as a param.

More streamlining is possible, but I'm trying to avoid
logic/typo bugs while fixing this. Eventually, we should
not need the `OperationData` class.


  Commit: c09be0d2a0f930a128c946329b42eef45d53062a
      https://github.com/llvm/llvm-project/commit/c09be0d2a0f930a128c946329b42eef45d53062a
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2021-01-20 (Wed, 20 Jan 2021)

  Changed paths:
    M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

  Log Message:
  -----------
  [SLP] reduce reduction code for checking vectorizable ops; NFC

This is another step towards removing `OperationData` and
fixing FMF matching/propagation bugs when forming reductions.


Compare: https://github.com/llvm/llvm-project/compare/6c1bc0d24cea...c09be0d2a0f9