[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)
commit92e625c2233c98a9ffa6d7d5cc10462957425054
tree6ce54c792b74472d825f50d9da479be6e336c388
parentddc2804e1a5451ee8f24ce93aaf98dbce29b5961
[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

llvm-svn: 364612
llvm/include/llvm/Analysis/InlineCost.h