No, it's 2% slower with it '#if 0'. I'm not going to commit in this state, I just wanted to show everybody how I'd verified that I'm maintaining the index list correctly and the general scheme of

LGTM2 http://codereview.appspot.com/206045/diff/1/4 File src/liboslexec/exec.cpp (right): http://codereview.appspot.com/206045/diff/1/4#newcode583 src/liboslexec/exec.cpp:583: #if 1 Should this be

LGTM http://codereview.appspot.com/206045/show

Incidentally, how much slower do we render by keeping both the runflags and indices in place? Only 2%. So this makes me think that quite soon into changing the individual shadeops, we'll break even

Reviewers: , Description: We've done extensive analysis showing that for typical shading batches we see in the wild, our method of having an array of true/false runflags is very wasteful, and despite

Solaris provides sincos as part of math.h. Given that OSL is now an open-source project, probably at some point it'll need to grow a configure based solution to find functions. If configure was

Here is the patch to fmath.h: Index: src/include/fmath.h =================================================================== --- src/include/fmath.h (revision 1392) +++ src/include/fmath.h (working

using portable sincos instead of calls to sin and cos http://codereview.appspot.com/205045/show

I found an alternative that gets around having to use _GNU_SOURCE (which might have other weird side effects). http://codereview.appspot.com/205045/show

1. At the top of fmath.h, before #include <cmath>: #ifndef __MATH_H__ #define _GNU_SOURCE #endif 2. In fmath.h, define sincos #if !defined(_GNU_SOURCE) || !defined(__GNUC__) ||

Of course, please substitute whatever compiler or version checks *correctly* discern whether sincos is available. -- Larry Gritz l...@...

Its more complicated than that. The symbol doesn't show up by default. You need to do: #define _GNU_SOURCE #include <math.h> And of course, its hard to know where math.h (or cmath) is first included

I would put in fmath.h: #ifndef __GNUC__ ... define it in terms of sin and cos... #endif -- Larry Gritz l...@...

You may This is a GNU extension, it isn't part of any standard to my knowledge. Can you think of a cross platform way of detecting it? http://codereview.appspot.com/205045/show

Dude, use the C++ sincos() function to compute the sines and cosines! You may as well make the underlying implementation faster in the same way. Other than that,

Reviewers: dev-osl_imageworks.com, osl-dev_googlegroups.com, Description: Add a new opcode that computes sin and cos together Please review this at

We should double-check that the placeholder struct symbol table entries don't take any space in the heap. They aren't intended to. If you don't have time to do it, please just file a ticket and I'll

I would have to check again, it seems to me the size was > 0. In any case, it seemed dangerous to give a valid heap address to symbols that should never be used. The nan fix is just that we were

LGTM http://codereview.appspot.com/204076/show

How much "space" were we allocating for the struct placeholders? Should be 0, no? Or was the problem just that check_nan was reading bogus heap areas to check nans on these placeholders? -- Larry