[all-commits] [llvm/llvm-project] 0e6318: Reapply "[Aarch64] [ISel] Don't save vaargs regist...

Pavel Skripkin via All-commits all-commits at lists.llvm.org
Mon Mar 24 08:14:05 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0e63180fa3e130c7da304578d29631c847fdcd62
      https://github.com/llvm/llvm-project/commit/0e63180fa3e130c7da304578d29631c847fdcd62
  Author: Pavel Skripkin <paskripkin at gmail.com>
  Date:   2025-03-24 (Mon, 24 Mar 2025)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    M llvm/test/CodeGen/AArch64/GlobalISel/aapcs_vararg_frame.ll
    M llvm/test/CodeGen/AArch64/alloca.ll
    M llvm/test/CodeGen/AArch64/darwinpcs-tail.ll
    M llvm/test/CodeGen/AArch64/vararg-tallcall.ll

  Log Message:
  -----------
  Reapply "[Aarch64] [ISel] Don't save vaargs registers if vaargs are unused" (#131459)

This reapplies original commit e122483762b44c7f4386165099ff2a404705d7d4

Second try, but with win64 removed from the scope, since it was somehow
broken by original commit.

### Original PR description

If vaargs are not used there is no need to save them. LLVM already
implements such optimization for x86, as well as gcc [1].

Some ABI tests are kept almost as-is, except for stack offsets, by just
adding llvm.va_start. Only laapcs_vararg_frame.ll test was rewritten to
match new behavior.

[1] https://godbolt.org/z/GWWKr8xMd



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