[all-commits] [llvm/llvm-project] 6ff91d: BranchFolder: Assert on SSA functions

Matt Arsenault via All-commits all-commits at lists.llvm.org
Wed Apr 27 15:51:50 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6ff91d17d66da46572e97f9a0b042182762cbe9e
      https://github.com/llvm/llvm-project/commit/6ff91d17d66da46572e97f9a0b042182762cbe9e
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2022-04-27 (Wed, 27 Apr 2022)

  Changed paths:
    M llvm/lib/CodeGen/BranchFolding.cpp

  Log Message:
  -----------
  BranchFolder: Assert on SSA functions

We probably should have the opposite of getRequiredProperties for this


  Commit: 717209763e17c10aaddce9db3faac4ecbf1afd29
      https://github.com/llvm/llvm-project/commit/717209763e17c10aaddce9db3faac4ecbf1afd29
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2022-04-27 (Wed, 27 Apr 2022)

  Changed paths:
    A llvm/test/tools/llvm-reduce/mir/preserve-mem-operands.mir
    M llvm/tools/llvm-reduce/ReducerWorkItem.cpp

  Log Message:
  -----------
  llvm-reduce: Fix incorrect cloning of MachineMemOperands

There were two problems with directly copying the MMOs from the old
function. The MMOs are owned by the function's Allocator, so need to
be reallocated anyways (surprisingly I didn't notice breakage on
this). Second, the PseudoSourceValues are also allocated per function
and need to be reallocated.


Compare: https://github.com/llvm/llvm-project/compare/7d2dddce8f0d...717209763e17


More information about the All-commits mailing list