[all-commits] [llvm/llvm-project] fc9bed: [RISCV][NFC] Test showing wrong scheduling of expa...

[Roger Ferrer Ibáñez via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fc9bed025e3328e0ae67fc762cd92b4de4ecc6c0
      https://github.com/llvm/llvm-project/commit/fc9bed025e3328e0ae67fc762cd92b4de4ecc6c0
  Author: Roger Ferrer Ibanez <roger.ferrer at bsc.es>
  Date:   2022-05-13 (Fri, 13 May 2022)

  Changed paths:
    A llvm/test/CodeGen/RISCV/rvv/wrong-chain-fixed-load.ll

  Log Message:
  -----------
  [RISCV][NFC] Test showing wrong scheduling of expansion of memmove for fixed RVV

After a memmove is expanded, we load the higher addresses after we have
stored onto the lower ones, as if we could do a forward copy.

Differential Revision: https://reviews.llvm.org/D125553


  Commit: 189ca6958e84272ced8b91011fc9b9b10ddbb99f
      https://github.com/llvm/llvm-project/commit/189ca6958e84272ced8b91011fc9b9b10ddbb99f
  Author: Roger Ferrer Ibanez <roger.ferrer at bsc.es>
  Date:   2022-05-13 (Fri, 13 May 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/test/CodeGen/RISCV/rvv/fixed-vectors-calling-conv.ll
    M llvm/test/CodeGen/RISCV/rvv/fixed-vectors-fp-vrgather.ll
    M llvm/test/CodeGen/RISCV/rvv/fixed-vectors-int-vrgather.ll
    M llvm/test/CodeGen/RISCV/rvv/wrong-chain-fixed-load.ll

  Log Message:
  -----------
  [RISCV] Use the new chain when converting a fixed RVV load

When building the final merged node, we were using the original chain
rather than the output chain of the new operation. After some collapsing
of the chain this could cause the loads be incorrectly scheduled respect
to later stores.

This was uncovered by SingleSource/Regression/C/gcc-c-torture/execute/pr36038.c
of the llvm testsuite.

https://reviews.llvm.org/D125560


Compare: https://github.com/llvm/llvm-project/compare/badd088c57d7...189ca6958e84