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

Matt Arsenault via All-commits all-commits at lists.llvm.org
Sat May 3 00:20:55 PDT 2025


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

  Changed paths:
    A llvm/test/CodeGen/ARM/shouldRewriteCopySrc.ll

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


  Commit: 0096a5d28395b183312cf7a43707bacdececfbf5
      https://github.com/llvm/llvm-project/commit/0096a5d28395b183312cf7a43707bacdececfbf5
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-05-03 (Sat, 03 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/c058c3ed03be...0096a5d28395

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