[all-commits] [llvm/llvm-project] a7a1b8: [MSSAUpdater] Handle simplified accesses when upda...

Nikita Popov via All-commits all-commits at lists.llvm.org
Wed Jan 24 01:15:55 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a7a1b8b17e264fb0f2d2b4165cf9a7f5094b08b3
      https://github.com/llvm/llvm-project/commit/a7a1b8b17e264fb0f2d2b4165cf9a7f5094b08b3
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2024-01-24 (Wed, 24 Jan 2024)

  Changed paths:
    M llvm/lib/Analysis/MemorySSAUpdater.cpp
    M llvm/test/Transforms/SimpleLoopUnswitch/memssa-readnone-access.ll

  Log Message:
  -----------
  [MSSAUpdater] Handle simplified accesses when updating phis (#78272)

This is a followup to #76819. After those changes, we can still run into
an assertion failure for a slight variation of the test case: When
fixing up MemoryPhis, we map the incoming access to the access of the
cloned instruction -- which may now no longer exist.

Fix this by reusing the getNewDefiningAccessForClone() helper, which
will look upwards for a new defining access in that case.




More information about the All-commits mailing list