[all-commits] [llvm/llvm-project] b77e55: [TailCallElim] Regenerate test checks with --funct...

[DianQK via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b77e5563f6bc4b5a81d427bf0f42ebea8ca376f0
      https://github.com/llvm/llvm-project/commit/b77e5563f6bc4b5a81d427bf0f42ebea8ca376f0
  Author: DianQK <dianqk at dianqk.net>
  Date:   2023-08-09 (Wed, 09 Aug 2023)

  Changed paths:
    M llvm/test/Transforms/TailCallElim/tre-byval-parameter-2.ll
    M llvm/test/Transforms/TailCallElim/tre-byval-parameter.ll

  Log Message:
  -----------
  [TailCallElim] Regenerate test checks with --function-signature (NFC)

For checking the readonly attribute.

Pre-commit test for D156793.

Reviewed By: nikic

Differential Revision: https://reviews.llvm.org/D156789


  Commit: c3f227ead65c606409ff8cc3333a6c751f156a9c
      https://github.com/llvm/llvm-project/commit/c3f227ead65c606409ff8cc3333a6c751f156a9c
  Author: DianQK <dianqk at dianqk.net>
  Date:   2023-08-09 (Wed, 09 Aug 2023)

  Changed paths:
    M llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
    A llvm/test/Transforms/PhaseOrdering/pr64289-tce.ll
    M llvm/test/Transforms/TailCallElim/tre-byval-parameter-2.ll
    M llvm/test/Transforms/TailCallElim/tre-byval-parameter.ll

  Log Message:
  -----------
  [TailCallElim] Remove the readonly attribute of byval.

When eliminating a tail call, we modify the values of the arguments.
Therefore, if the byval parameter has a readonly attribute, we have to remove it. It is safe because,
from the perspective of a caller, the byval parameter is always treated as "readonly," even if the readonly attribute is removed.

Fixes #64289.

Reviewed By: nikic

Differential Revision: https://reviews.llvm.org/D156793


Compare: https://github.com/llvm/llvm-project/compare/5b95d17da1f0...c3f227ead65c