[all-commits] [llvm/llvm-project] 3f1e42: Cleanup: addressing review.

[Paschalis Mpeis via All-commits]

  Branch: refs/heads/users/paschalis-mpeis/replace-with-veclip-scalable-vec
  Home:   https://github.com/llvm/llvm-project
  Commit: 3f1e423f3a06410d7a9f787f5d1a3f89453f3b57
      https://github.com/llvm/llvm-project/commit/3f1e423f3a06410d7a9f787f5d1a3f89453f3b57
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/include/llvm/Analysis/VectorUtils.h
    M llvm/lib/Analysis/VectorUtils.cpp
    M llvm/unittests/Analysis/VectorFunctionABITest.cpp

  Log Message:
  -----------
  Cleanup: addressing review.


  Commit: 3f9626b8b2ea7841e517bb855479e66839ed0d06
      https://github.com/llvm/llvm-project/commit/3f9626b8b2ea7841e517bb855479e66839ed0d06
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/test/Transforms/LoopVectorize/AArch64/sleef-intrinsic-calls-aarch64.ll

  Log Message:
  -----------
  [NFC][TLI] Improve tests for ArmPL and SLEEF Intrinsics.

Auto-generate test `armpl-intrinsics.ll`, and use active lane mask to
have shorter `shufflevector` check lines.

Update scripts now add `@llvm.compiler.used` instead of using the regex:
`@[[LLVM_COMPILER_USED:[a-zA-Z0-9_$"\\.-]+]]`


  Commit: 50e0cb7ea4cc7b99c8f15565a01598e00f07962a
      https://github.com/llvm/llvm-project/commit/50e0cb7ea4cc7b99c8f15565a01598e00f07962a
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/test/Transforms/LoopVectorize/AArch64/sleef-intrinsic-calls-aarch64.ll

  Log Message:
  -----------
  Add `simplifycfg` pass and `noalias` to ensure tail folding.

`noalias` attribute was added only to the `%in.ptr` parameter of the
ArmPL Intrinsics.


  Commit: 83719090324493b78537e74715413bdf058a9eaf
      https://github.com/llvm/llvm-project/commit/83719090324493b78537e74715413bdf058a9eaf
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/lib/Analysis/VFABIDemangling.cpp
    M llvm/lib/CodeGen/ReplaceWithVeclib.cpp
    M llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-armpl.ll
    M llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-sleef-scalable.ll

  Log Message:
  -----------
  [TLI] Pass replace-with-veclib works with Scalable Vectors.

The pass uses the Masked variant of TLI method when the Intrinsic
operates on Scalable Vectors and it fails to find a non-Masked variant.


  Commit: b24f0e595f21bbaa17b414c02d3641d9a6422e54
      https://github.com/llvm/llvm-project/commit/b24f0e595f21bbaa17b414c02d3641d9a6422e54
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/lib/CodeGen/ReplaceWithVeclib.cpp
    M llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-armpl.ll
    M llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-sleef-scalable.ll

  Log Message:
  -----------
  Use createFunctionType to correctly replace veclib calls.

Split replaceWithTLIFunction method into two methods.


  Commit: bfcf1869e6595a8be3be7aa9aab371e56b8b86b4
      https://github.com/llvm/llvm-project/commit/bfcf1869e6595a8be3be7aa9aab371e56b8b86b4
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2023-12-14 (Thu, 14 Dec 2023)

  Changed paths:
    M llvm/lib/CodeGen/ReplaceWithVeclib.cpp
    M llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-sleef-scalable.ll

  Log Message:
  -----------
  getVecDesc now prioritizes masked variant

Also further cleanup to address reviewers.


Compare: https://github.com/llvm/llvm-project/compare/34848d0cb1ee...bfcf1869e659