Bug 52003 - Add support for OES_standard_derivatives in GLSL ES
Summary: Add support for OES_standard_derivatives in GLSL ES
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: glsl-compiler (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Ian Romanick
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-12 09:04 UTC by Tomeu Vizoso
Modified: 2012-08-02 06:09 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
patch (2.54 KB, patch)
2012-07-12 09:06 UTC, Tomeu Vizoso
Details | Splinter Review

Description Tomeu Vizoso 2012-07-12 09:04:40 UTC
When using GLES2, glGetString(GL_EXTENSIONS) advertises GL_OES_standard_derivatives but the functions added by that extension are only made available to GLSL >= 1.10, without regard to GL ES or GL.
Comment 1 Tomeu Vizoso 2012-07-12 09:06:11 UTC
Created attachment 64136 [details] [review]
patch

Add support for OES_standard_derivatives in GLSL ES

Previously, we advertised the extension but the builtin functions
were enabled only for GLSL and not for ES.
Comment 2 Ian Romanick 2012-07-16 17:23:28 UTC
The patch looks good.  Can you make a couple minor tweaks to the commit message and send it to the mesa-dev mailing list?  Specifically,

1. Add a Signed-off-by line.

2. List the bug on its own like as:

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=52003

3. Add my Reviewed-by:

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>

Thanks!
Comment 3 Tomeu Vizoso 2012-07-18 09:26:26 UTC
(In reply to comment #2)
> The patch looks good.  Can you make a couple minor tweaks to the commit message
> and send it to the mesa-dev mailing list?  Specifically,
> 
> 1. Add a Signed-off-by line.
> 
> 2. List the bug on its own like as:
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=52003
> 
> 3. Add my Reviewed-by:
> 
> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
> 
> Thanks!

Done, anything else I should do about this?

http://comments.gmane.org/gmane.comp.video.mesa3d.devel/42624
Comment 4 Tomeu Vizoso 2012-07-18 20:00:28 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > The patch looks good.  Can you make a couple minor tweaks to the commit message
> > and send it to the mesa-dev mailing list?  Specifically,
> > 
> > 1. Add a Signed-off-by line.
> > 
> > 2. List the bug on its own like as:
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=52003
> > 
> > 3. Add my Reviewed-by:
> > 
> > Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
> > 
> > Thanks!
> 
> Done, anything else I should do about this?
> 
> http://comments.gmane.org/gmane.comp.video.mesa3d.devel/42624

Forgot to mention that though I have an account in fd.o, I don't have push rights in mesa's repo so I will need someone to push it for me. Thanks!
Comment 5 Tomeu Vizoso 2012-08-02 06:09:04 UTC
Kenneth pushed this one. Thanks!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.