[all-commits] [llvm/llvm-project] bc48a2: [LoopPeel] Use reference instead of pointer for DT...

[annamthomas via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bc48a266554740bbedbc155deda2aa81b9a1602d
      https://github.com/llvm/llvm-project/commit/bc48a266554740bbedbc155deda2aa81b9a1602d
  Author: Anna Thomas <anna at azul.com>
  Date:   2022-02-01 (Tue, 01 Feb 2022)

  Changed paths:
    M llvm/include/llvm/Transforms/Utils/LoopPeel.h
    M llvm/lib/Transforms/Scalar/LoopFuse.cpp
    M llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
    M llvm/lib/Transforms/Utils/LoopPeel.cpp

  Log Message:
  -----------
  [LoopPeel] Use reference instead of pointer for DT argument

Cleanup code in peelLoop API. We already have usage of DT without guarding
against a null DT, so this change constant folds the remaining null DT
checks.
Also make the argument a reference so that it is clear the argument is
a nonnull DT.
Extracted from D118472.


  Commit: f50821cff048209bfb98991b978c87cf24d37904
      https://github.com/llvm/llvm-project/commit/f50821cff048209bfb98991b978c87cf24d37904
  Author: Anna Thomas <anna at azul.com>
  Date:   2022-02-01 (Tue, 01 Feb 2022)

  Changed paths:
    M llvm/lib/Transforms/Scalar/LoopFuse.cpp

  Log Message:
  -----------
  [LoopFuse] Add assertion for non-null DT in fusion candidate

The code paths analyzed (all constructor invocations of fusion
candidate) pass in a non-null DT.
Adding this assert as requested in D118472 before converting this to a
reference argument.


Compare: https://github.com/llvm/llvm-project/compare/453620f55ea3...f50821cff048