[all-commits] [llvm/llvm-project] e74102: [CostModel][X86] Merge getTypeBasedIntrinsicInstrC...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Wed Sep 7 04:04:38 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e74102a9637edc65baa1a391031a9791c147c3e2
      https://github.com/llvm/llvm-project/commit/e74102a9637edc65baa1a391031a9791c147c3e2
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2022-09-07 (Wed, 07 Sep 2022)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/lib/Target/X86/X86TargetTransformInfo.h
    M llvm/test/Analysis/CostModel/X86/cttz.ll

  Log Message:
  -----------
  [CostModel][X86] Merge getTypeBasedIntrinsicInstrCost into getIntrinsicInstrCost

For the few non type based intrinsic cases we can just check for !isTypeBasedOnly() to access the args directly.

I don't think we have a need to keep getTypeBasedIntrinsicInstrCost in BasicTTIImpl.h any more and can do a similar merge there as well - but it's a messier refactor and will take a while.




More information about the All-commits mailing list