[all-commits] [llvm/llvm-project] d02137: ARM: Add test which shows overriding shouldRewrite...

Matt Arsenault via All-commits all-commits at lists.llvm.org
Mon May 5 09:09:58 PDT 2025


  Branch: refs/heads/users/arsenm/arm/remove-shouldRewriteCopySrc
  Home:   https://github.com/llvm/llvm-project
  Commit: d021375367614a699c84c4e0d793d4e4b995ba72
      https://github.com/llvm/llvm-project/commit/d021375367614a699c84c4e0d793d4e4b995ba72
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-05-05 (Mon, 05 May 2025)

  Changed paths:

  Log Message:
  -----------
  ARM: Add test which shows overriding shouldRewriteCopySrc does something


  Commit: 96884a6427257c67618296ce89da6cdb7280a479
      https://github.com/llvm/llvm-project/commit/96884a6427257c67618296ce89da6cdb7280a479
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-05-05 (Mon, 05 May 2025)

  Changed paths:
    M llvm/lib/Target/ARM/ARMBaseRegisterInfo.cpp
    M llvm/lib/Target/ARM/ARMBaseRegisterInfo.h
    M llvm/test/CodeGen/ARM/shouldRewriteCopySrc.ll

  Log Message:
  -----------
  ARM: Remove override of shouldRewriteCopySrc

All of the overrides of shouldRewriteCopySrc appear to be hacks
for bugs in the base implementation, so I'm trying to delete
all of the overrides. I was expecting this to find an example
issue like the x86 version, but no tests change with this.


Compare: https://github.com/llvm/llvm-project/compare/0096a5d28395...96884a642725

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list