[all-commits] [llvm/llvm-project] d439dd: [clang] Add more test coverage for "zero-call-used...

[Lucas Chollet via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d439dde4e6c0af6126ccb6c222653bfce07ad3d1
      https://github.com/llvm/llvm-project/commit/d439dde4e6c0af6126ccb6c222653bfce07ad3d1
  Author: Lucas Chollet <lucas.chollet at serenityos.org>
  Date:   2026-06-09 (Tue, 09 Jun 2026)

  Changed paths:
    M clang/test/CodeGen/zero-call-used-regs.c
    M llvm/test/CodeGen/AArch64/zero-call-used-regs.ll
    M llvm/test/CodeGen/X86/zero-call-used-regs.ll

  Log Message:
  -----------
  [clang] Add more test coverage for "zero-call-used-regs" (#201834)

In clang, the main function is treated as a special case and no
"zero-call-used-regs" attributes are emitted for it. This patch adds
proper unit testing for that functionality.

AFAIU, treating `main` as a special case was initially implemented in
the backend. However, during review [1] the logic was moved to clang
while the associated test case was never updated (and then copied over
to the AArch64 test file).

[1] https://reviews.llvm.org/D110869
More specifically this comment: https://reviews.llvm.org/D110869#3285932

---

Commenting out
https://github.com/llvm/llvm-project/blob/8e217eda17df80d2035d6ff27e74bfa8df77339a/clang/lib/CodeGen/CodeGenFunction.cpp#L1102-L1104
makes the new tests fail while the removed test still pass.



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