[all-commits] [llvm/llvm-project] 984596: [MergedLoadStoreMotion] Add tests for store withou...

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9845969198cf41eecafdb8f55b2ae85b41aa237d
      https://github.com/llvm/llvm-project/commit/9845969198cf41eecafdb8f55b2ae85b41aa237d
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2022-12-27 (Tue, 27 Dec 2022)

  Changed paths:
    A llvm/test/Transforms/MergedLoadStoreMotion/st_sink_no_gep.ll

  Log Message:
  -----------
  [MergedLoadStoreMotion] Add tests for store without GEPs (NFC)

MergedLoadStoreMotion currently only handles the case where each
store has it's own GEP. It fails to handle the case where the
store argument is exactly the same.


  Commit: 2c15b9d9e1a898cfd849db81b36d278eac3ef24e
      https://github.com/llvm/llvm-project/commit/2c15b9d9e1a898cfd849db81b36d278eac3ef24e
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2022-12-27 (Tue, 27 Dec 2022)

  Changed paths:
    M llvm/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
    M llvm/test/Transforms/MergedLoadStoreMotion/st_sink_no_gep.ll
    M llvm/test/Transforms/MergedLoadStoreMotion/st_sink_split_bb.ll

  Log Message:
  -----------
  [MergeLoadStoreMotion] Don't require GEP for sinking

Allow sinking stores where both operands are the same, don't require
them to have an identical GEP in each block.

This came up when migrating tests to opaque pointers, where
zero-index GEPs are omitted.


Compare: https://github.com/llvm/llvm-project/compare/282c1e362be5...2c15b9d9e1a8