[all-commits] [llvm/llvm-project] fb8038: [TTI] getExtendedReductionCost - replace std::opti...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Thu Apr 13 03:26:47 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fb8038db73f893c268ab69b17c1b5b193d40797d
      https://github.com/llvm/llvm-project/commit/fb8038db73f893c268ab69b17c1b5b193d40797d
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2023-04-13 (Thu, 13 Apr 2023)

  Changed paths:
    M llvm/include/llvm/Analysis/TargetTransformInfo.h
    M llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/lib/Analysis/TargetTransformInfo.cpp
    M llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
    M llvm/lib/Target/ARM/ARMTargetTransformInfo.h
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h

  Log Message:
  -----------
  [TTI] getExtendedReductionCost - replace std::optional<FastMathFlags> args with FastMathFlags

Followup to D148149 where it was noticed that the std::optional wrapper wasn't helping with anything (we can just use an empty FastMathFlags()).


  Commit: 1afbdbd27d6b65649af23475c5f530ff15c736af
      https://github.com/llvm/llvm-project/commit/1afbdbd27d6b65649af23475c5f530ff15c736af
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2023-04-13 (Thu, 13 Apr 2023)

  Changed paths:
    M llvm/include/llvm/Analysis/TargetTransformInfo.h

  Log Message:
  -----------
  [TTI] Remove unnecessary default CostKind args from getExtendedReductionCost/getMulAccReductionCost wrappers. NFC.

We should only ever call these from the TargetTransformInfo interface, which provides all the args.


Compare: https://github.com/llvm/llvm-project/compare/2524000187fc...1afbdbd27d6b


More information about the All-commits mailing list