[all-commits] [llvm/llvm-project] 29fada: [EarlyCSE] Don't eagerly optimize MemoryUses

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 29fada4a3d3db309f11f7fa7a0c61cd4021e9947
      https://github.com/llvm/llvm-project/commit/29fada4a3d3db309f11f7fa7a0c61cd4021e9947
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2022-03-23 (Wed, 23 Mar 2022)

  Changed paths:
    M llvm/lib/Transforms/Scalar/EarlyCSE.cpp

  Log Message:
  -----------
  [EarlyCSE] Don't eagerly optimize MemoryUses

EarlyCSE currently optimizes all MemoryUses upfront. However,
EarlyCSE only actually queries the clobbering memory access for
a subset of uses, namely those where a CSE candidate has already
been identified. Delaying use optimization to the clobber query
improves compile-time in practice.

This change is not NFC because EarlyCSE has a limit on the number
of clobber queries (EarlyCSEMssaOptCap), in which case it falls
back to the defining access. The defining access for uses will now
no longer coincide with the optimized access.

If there are performance regressions from this change, we should
be able to address them by raising this limit.

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