[all-commits] [llvm/llvm-project] 664d0c: [TargetTransformInfo] move branch probability quer...

[RotateRight via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 664d0c052c3154c7a9d0cfbce7b23b4bd0b16841
      https://github.com/llvm/llvm-project/commit/664d0c052c3154c7a9d0cfbce7b23b4bd0b16841
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2021-03-22 (Mon, 22 Mar 2021)

  Changed paths:
    M llvm/include/llvm/Analysis/TargetTransformInfo.h
    M llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
    M llvm/include/llvm/CodeGen/TargetLowering.h
    M llvm/lib/Analysis/TargetTransformInfo.cpp
    M llvm/lib/CodeGen/CodeGenPrepare.cpp
    M llvm/lib/CodeGen/TargetLoweringBase.cpp
    M llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp

  Log Message:
  -----------
  [TargetTransformInfo] move branch probability query from TargetLoweringInfo

This is no-functional-change intended (NFC), but needed to allow
optimizer passes to use the API. See D98898 for a proposed usage
by SimplifyCFG.

I'm simplifying the code by removing the cl::opt. That was added
back with the original commit in D19488, but I don't see any
evidence in regression tests that it was used. Target-specific
overrides can use the usual patterns to adjust as necessary.
We could also restore that cl::opt, but it was not clear to me
exactly how to do it in the convoluted TTI class structure.


  Commit: c21016715f0ee4a36affdf7150ac135ca98b0eae
      https://github.com/llvm/llvm-project/commit/c21016715f0ee4a36affdf7150ac135ca98b0eae
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2021-03-22 (Mon, 22 Mar 2021)

  Changed paths:
    M llvm/test/Transforms/SimplifyCFG/preserve-branchweights.ll

  Log Message:
  -----------
  [SimplifyCFG] adjust test branchweights; NFC

This will check the boundary conditions of the
revised change proposed in D98898.


Compare: https://github.com/llvm/llvm-project/compare/fe5f66d925c3...c21016715f0e