search for: r256521

Hi, After r256521 - which removes InlineCostAnalysis class - I think there is no strong reason for InlineCost.cpp to be part of the Analysis library. Is it fine to make it part of TransformUtils? I submitted r266477 (which has now been reverted) that made Analysis depend on ProfileData in order to obtain ProfileSu...

...t;, "Philip Reames" >> <listmail at philipreames.com>, "David Li" <davidxl at google.com> >> Sent: Monday, April 18, 2016 2:39:49 PM >> Subject: Move InlineCost.cpp out of Analysis? >> >> >> Hi, >> >> >> After r256521 - which removes InlineCostAnalysis class - I think >> there is no strong reason for InlineCost.cpp to be part of the >> Analysis library. Is it fine to make it part of TransformUtils? >> > > Given that InlineCost is not really an analysis any longer, I think this is fine....

...com>, "David Li" <davidxl at google.com> >> >> Sent: Monday, April 18, 2016 2:39:49 PM >> >> Subject: Move InlineCost.cpp out of Analysis? >> >> >> >> >> >> Hi, >> >> >> >> >> >> After r256521 - which removes InlineCostAnalysis class - I think >> >> there is no strong reason for InlineCost.cpp to be part of the >> >> Analysis library. Is it fine to make it part of TransformUtils? >> >> >> > >> > Given that InlineCost is not really an...

...t;davidxl at google.com> >>> >> Sent: Monday, April 18, 2016 2:39:49 PM >>> >> Subject: Move InlineCost.cpp out of Analysis? >>> >> >>> >> >>> >> Hi, >>> >> >>> >> >>> >> After r256521 - which removes InlineCostAnalysis class - I think >>> >> there is no strong reason for InlineCost.cpp to be part of the >>> >> Analysis library. Is it fine to make it part of TransformUtils? >>> >> >>> > >>> > Given that InlineCo...

...> > > > > > >> > > > > > > > > > > >> Hi, > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> After r256521 - which removes InlineCostAnalysis class - I > > > > >> think > > > > > > > > > > >> there is no strong reason for InlineCost.cpp to be part of > > > > >> the > > > > > > > > > > >> Ana...