[all-commits] [llvm/llvm-project] 032259: [NFC] Rename LimitingIntrinsic into MinMaxIntrinsic

[Roman Lebedev via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 03225969e39ce3ab88673cd047a2a546d655dd95
      https://github.com/llvm/llvm-project/commit/03225969e39ce3ab88673cd047a2a546d655dd95
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-04-10 (Sat, 10 Apr 2021)

  Changed paths:
    M llvm/include/llvm/IR/IntrinsicInst.h

  Log Message:
  -----------
  [NFC] Rename LimitingIntrinsic into MinMaxIntrinsic

As requested in post-commit review


  Commit: 257eda07940d8e6d46dbb5944589a0186e07c6e9
      https://github.com/llvm/llvm-project/commit/257eda07940d8e6d46dbb5944589a0186e07c6e9
  Author: Roman Lebedev <lebedev.ri at gmail.com>
  Date:   2021-04-10 (Sat, 10 Apr 2021)

  Changed paths:
    M llvm/include/llvm/Analysis/LazyValueInfo.h
    M llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
    M llvm/lib/Transforms/Scalar/JumpThreading.cpp

  Log Message:
  -----------
  [NFC][LVI] getPredicateAt(): drop default value for UseBlockValue

The default is likely wrong.
Out of all the callees, only a single one needs to pass-in false (JumpThread),
everything else either already passes true, or should pass true.

Until the default is flipped, at least make it harder to unintentionally
add new callees with UseBlockValue=false.


Compare: https://github.com/llvm/llvm-project/compare/8f16101c703e...257eda07940d