[all-commits] [llvm/llvm-project] 895805: [ArgPromotion] Move isDenselyPacked static member ...

[Pavel Samolysov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8958057fb1009ee3ef6be1d509ddc4fe801add42
      https://github.com/llvm/llvm-project/commit/8958057fb1009ee3ef6be1d509ddc4fe801add42
  Author: Pavel Samolysov <samolisov at gmail.com>
  Date:   2022-06-29 (Wed, 29 Jun 2022)

  Changed paths:
    M llvm/include/llvm/Transforms/IPO/ArgumentPromotion.h
    M llvm/lib/Transforms/IPO/ArgumentPromotion.cpp
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp

  Log Message:
  -----------
  [ArgPromotion] Move isDenselyPacked static member (NFC)

The `isDenselyPacked` static member of the `ArgumentPromotionPass` class
is not used in the class itself anymore. The single known user of the
function is in the `AttributorAttributes.cpp` file, so the function has
been moved into the file.

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


  Commit: 3d9ce9e43d07bbd3ab5c25ee36f417e3d372b94b
      https://github.com/llvm/llvm-project/commit/3d9ce9e43d07bbd3ab5c25ee36f417e3d372b94b
  Author: Pavel Samolysov <samolisov at gmail.com>
  Date:   2022-06-29 (Wed, 29 Jun 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/ArgumentPromotion.cpp

  Log Message:
  -----------
  [ArgPromotion] Remove all the getters and ReplaceCallSite (NFC)

AARGetter is an abstraction over a source of the `AAResults` introduced
to support the legacy pass manager as well as the modern one. Since the
Argument Promotion pass doesn't support the legacy pass manager anymore,
the abstraction is not required and `AAResults` may be used directly.

The instance of the `FunctionAnalysisManager` is passed through the
functions to get all the required analyses just wherever they are
required and do not use the awkward getter callbacks.

The `ReplaceCallSite` parameter was required for the legacy pass manager
only and isn't used anymore, so the parameter has been eliminated.

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


Compare: https://github.com/llvm/llvm-project/compare/0a6e29f455ff...3d9ce9e43d07