https://codereview.appspot.com/547980044/diff/567490043/lily/include/skyline.hh
File lily/include/skyline.hh (right):

https://codereview.appspot.com/547980044/diff/567490043/lily/include/skyline.hh#newcode152
lily/include/skyline.hh:152: }
On 2020/04/24 17:12:48, hanwenn wrote:
> On 2020/04/24 16:33:04, hahnjo wrote:
> > Why do you need all of this in the header file? As far as I can see,
nobody
> else
> > is calling these methods, so the argument of inlinining does not
apply.
> 
> trimmed this a bit.

looks like GCC doesn't want to inline the functions if I do that,
leading to significant slowdown.

https://codereview.appspot.com/547980044/

Reply via email to