[all-commits] [llvm/llvm-project] ba5b01: [InlineCost] CallAnalyzer: use TTI info for extrac...

Roman Lebedev via All-commits all-commits at lists.llvm.org
Fri Apr 30 03:55:43 PDT 2021


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

  Changed paths:
    M llvm/lib/Analysis/InlineCost.cpp
    M llvm/test/Transforms/Inline/X86/extractvalue.ll

  Log Message:
  -----------
  [InlineCost] CallAnalyzer: use TTI info for extractvalue - they are free (PR50099)

It seems incorrect to use TTI data in some places,
and override it in others. In this case, TTI says
that `extractvalue` are free, yet we bill them.

While this doesn't address https://bugs.llvm.org/show_bug.cgi?id=50099 yet,
it reduces the cost from 55 to 50 while the threshold is 45.

Differential Revision: https://reviews.llvm.org/D101228




More information about the All-commits mailing list