]> granicus.if.org Git - llvm/commit
[InlineCost] make InlineCost assignable
authorFedor Sergeev <fedor.sergeev@azul.com>
Thu, 27 Jun 2019 23:41:03 +0000 (23:41 +0000)
committerFedor Sergeev <fedor.sergeev@azul.com>
Thu, 27 Jun 2019 23:41:03 +0000 (23:41 +0000)
commita7789c6bb3ee54f194ad10fd697ebf490591a0e2
tree44d3e307301f90d416815712c1eb354a1bd0d9e8
parentcf66538c755246bc7df4b1a93a4c667e3a33618e
[InlineCost] make InlineCost assignable

Summary:
Current InlineCost is not assignable because of const members Cost and Threshold.
I dont see practical benefits from having them const (access to these members is
private and internal interactions are rather simple). On other hand that makes
it hard to use as a member in some other data structure where assignability is necessary.

I'm going to use InlineCost in a downstream inliner that maintains a complex queue
of candidate call-sites and thus keeping and recalculating InlineCost is necessary.

This patch just removes 'const' from both members, making InlineCost assignable.

Reviewers: eraman, greened, chandlerc, yrouban, apilipenko
Reviewed By: apilipenko
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63823

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@364612 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/Analysis/InlineCost.h