[RFC PATCH] gas model: use virial cubic polynomial form
Dirk Hohndel
dirk at hohndel.org
Fri Mar 4 10:17:07 PST 2016
On Thu, Mar 03, 2016 at 09:46:19PM -0800, Linus Torvalds wrote:
>
> So for this to make sense, we'd need to feel that performance matters. We
> *do* end up doing the gas use for each plot point, so it's not like this
> function might not get called a lot.
>
> But compiler options will change the code generation even more than this
> ever will, and aggressive compiler optimizations would have made the old
> code better too (ie just inlining and CSE on the pressure power series
> would do good things to the thing without this patch too).
>
> In other words: I think this patch is correct, and it *does* make the code
> faster. But do we even care at this point?
Obviously, fewer computations are good. I semi-concur with Robert that we
are over engineering this, but I think the rationale of using the virial
form makes sense. So I'm inclined to take this patch as it both uses a
more reasonable formula AND reduces computational complexity.
/D
More information about the subsurface
mailing list