From: bugzilla-daemon@freedesktop.org
To: dri-devel@lists.freedesktop.org
Subject: [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.
Date: Sat, 30 Apr 2016 01:11:16 +0000 [thread overview]
Message-ID: <bug-95085-502-ZASrkMHzb4@http.bugs.freedesktop.org/> (raw)
In-Reply-To: <bug-95085-502@http.bugs.freedesktop.org/>
[-- Attachment #1.1: Type: text/plain, Size: 1395 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=95085
--- Comment #15 from Roland Scheidegger <sroland@vmware.com> ---
(In reply to Nicolai Hähnle from comment #14)
> The trace uses glVertexAttribPointer instead of glVertexAttribIPointer to
> set up a vertex element that will consumed by an uint GLSL variable. I kind
> of suspect that that should be undefined behaviour, but I haven't found a
> corresponding spec reference.
I can't see how that could work (I don't understand how that could work with
some drivers even). glVertexAttribPointer() means the data is really floats -
if you'd use a non-float type to specify it, it will implicitly get converted
to floats. So ok you could cheat there which would leave the data untouched (I
haven't looked at the trace), and rely on the uint input to just read that data
as-is maybe. Not sure if that could work even if you could trick the
implementation to do this - integers as floats have some bad habits, namely all
your negative numbers tend to change their value (as they are NaNs), and your
positive numbers may have some attraction to zero (as they are denorms...).
In any case, just don't do this - these different attrib pointers exist for a
reason. Also see
http://stackoverflow.com/questions/28014864/why-different-variations-of-glvertexattribpointer-do-exist
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 2491 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-04-30 1:11 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <bug-95085-502@http.bugs.freedesktop.org/>
2016-04-25 14:35 ` [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters bugzilla-daemon
2016-04-29 18:55 ` bugzilla-daemon
2016-04-29 21:07 ` bugzilla-daemon
2016-04-29 23:27 ` bugzilla-daemon
2016-04-30 1:11 ` bugzilla-daemon [this message]
2016-04-30 1:20 ` bugzilla-daemon
2016-04-30 18:51 ` bugzilla-daemon
2016-05-01 17:32 ` bugzilla-daemon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bug-95085-502-ZASrkMHzb4@http.bugs.freedesktop.org/ \
--to=bugzilla-daemon@freedesktop.org \
--cc=dri-devel@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.