[all-commits] [llvm/llvm-project] c1cf51: [Polly][OpTree] Better report applied changes.

[Michael Kruse via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c1cf51e77701719f3439eed148dfaf44e726c586
      https://github.com/llvm/llvm-project/commit/c1cf51e77701719f3439eed148dfaf44e726c586
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2020-11-11 (Wed, 11 Nov 2020)

  Changed paths:
    M polly/lib/Transform/ForwardOpTree.cpp
    M polly/test/ForwardOpTree/forward_readonly.ll

  Log Message:
  -----------
  [Polly][OpTree] Better report applied changes.

Print to dbgs() any taken action.

Also, read-only scalars do not require any action unless
-polly-analyze-read-only-scalars=true is used. Better refect this by
using ForwardingAction::triviallyForwardable and thus not bumping the
statistics.


  Commit: c8a0e27cfb8311cdacbc6509cb3a7bf4e055affd
      https://github.com/llvm/llvm-project/commit/c8a0e27cfb8311cdacbc6509cb3a7bf4e055affd
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2020-11-11 (Wed, 11 Nov 2020)

  Changed paths:
    M polly/lib/Transform/ForwardOpTree.cpp
    A polly/test/ForwardOpTree/changed-kind.ll

  Log Message:
  -----------
  [Polly][OpTree] Fix mid-processing change of access kind.

Operand tree forwarding can cause the change of an access kind; in
particular change from a scalar kind to an array kind if the scalar
dependency is not necessary. Such an access cannot and doesn't need to
be forwarded anymore.

Fixes llvm.org/PR48034


Compare: https://github.com/llvm/llvm-project/compare/4c55c3b66dea...c8a0e27cfb83