Ian Romanick [Thu, 14 Oct 2010 20:28:42 +0000 (13:28 -0700)]
glsl: Slightly change the semantic of _LinkedShaders
Previously _LinkedShaders was a compact array of the linked shaders
for each shader stage. Now it is arranged such that each slot,
indexed by the MESA_SHADER_* defines, refers to a specific shader
stage. As a result, some slots will be NULL. This makes things a
little more complex in the linker, but it simplifies things in other
places.
As a side effect _NumLinkedShaders is removed.
NOTE: This may be a candidate for the 7.9 branch. If there are other
patches that get backported to 7.9 that use _LinkedShader, this patch
should be cherry picked also.
Eric Anholt [Fri, 15 Oct 2010 00:01:06 +0000 (17:01 -0700)]
i965: Fix texturing on pre-gen5.
I broke it in
06fd639c519214b6ebcbf29127b6d9ed429f8641 by only testing
2 generations of hardware :(
Brian Paul [Thu, 14 Oct 2010 22:31:54 +0000 (16:31 -0600)]
llvmpipe: code to dump bytecode to file (disabled)
Brian Paul [Thu, 14 Oct 2010 22:21:32 +0000 (16:21 -0600)]
gallivm: add compile-time option to emit inst addrs and/or line numbers
Disabling address printing is helpful for diffing.
Brian Paul [Thu, 14 Oct 2010 20:03:46 +0000 (14:03 -0600)]
mesa: remove post-convolution width/height vars
These were left-over bits from when convolution was removed.
Kenneth Graunke [Thu, 14 Oct 2010 20:41:29 +0000 (13:41 -0700)]
glsl: Refresh autogenerated file builtin_function.cpp.
Kenneth Graunke [Thu, 14 Oct 2010 21:33:30 +0000 (14:33 -0700)]
glsl: Add support for the 1.30 round() built-in.
This implements round() via the ir_unop_round_even opcode, rather than
adding a new opcode. We may wish to add one in the future, since it
might enable a small performance increase on some hardware, but for now,
this should suffice.
Kenneth Graunke [Thu, 14 Oct 2010 21:31:54 +0000 (14:31 -0700)]
i965: Add support for ir_unop_round_even via the RNDE instruction.
Kenneth Graunke [Thu, 14 Oct 2010 20:40:26 +0000 (13:40 -0700)]
glsl: Add front-end support for GLSL 1.30's roundEven built-in.
Implemented using the op-code introduced in the previous commit.
Kenneth Graunke [Thu, 14 Oct 2010 20:37:03 +0000 (13:37 -0700)]
glsl: Add a new ir_unop_round_even opcode for GLSL 1.30's roundEven.
Also, update ir_to_mesa's "1.30 is unsupported" case to "handle" it.
Dave Airlie [Thu, 14 Oct 2010 22:46:16 +0000 (08:46 +1000)]
r300g: clean up warning due to unknown cap.
Keith Whitwell [Thu, 14 Oct 2010 16:19:00 +0000 (17:19 +0100)]
r600g: handle absolute modifier in shader translator
This was being classed as unsupported in one place but used in others.
Enabling it seems to work fine.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Keith Whitwell [Thu, 14 Oct 2010 15:42:39 +0000 (16:42 +0100)]
r600g: emit hardware linewidth
Tested with demos/pixeltest - line rasterization doesn't seem to be
set up for GL conventions yet, but at least width is respected now.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Keith Whitwell [Thu, 14 Oct 2010 15:42:38 +0000 (16:42 +0100)]
r600/drm: fix segfaults in winsys create failure path
Would try to destroy radeon->cman, radeon->kman both which were still
NULL.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Kenneth Graunke [Thu, 14 Oct 2010 20:19:23 +0000 (13:19 -0700)]
i965: Clean up a warning in the old fragment backend.
Hopefully this code can just go away soon.
Eric Anholt [Thu, 14 Oct 2010 19:23:29 +0000 (12:23 -0700)]
i965: Enable the new FS backend on pre-gen6 as well.
It is now to the point where we have no regressing piglit tests. It
also fixes Yo Frankie! and Humus DynamicBranching, probably due to the
piglit bias tests that work that didn't on the Mesa IR backend.
As a downside, performance takes about a 5-10% performance hit at the
moment (e.g. nexuiz 19.8fps -> 18.8fps), which I plan to resolve by
reintroducing 16-wide fragment shaders where possible. It is a win,
though, for fragment shaders using flow control.
Kenneth Graunke [Thu, 14 Oct 2010 18:40:19 +0000 (11:40 -0700)]
i965: Correctly emit the RNDZ instruction.
Simply using RNDU, RNDZ, or RNDE does not produce the desired result.
Rather, the RND* instructions place a value in the destination register
that may be 1 less than the correct answer. They can also set per-channel
"increment bits" in a flag register, which, if set, mean dest needs to
be incremented by 1. A second instruction - a predicated add -
completes the job.
Notably, RNDD always produces the correct answer in a single
instruction.
Fixes piglit test glsl-fs-trunc.
Kenneth Graunke [Thu, 14 Oct 2010 18:39:09 +0000 (11:39 -0700)]
i965: Use RNDZ for ir_unop_trunc in the new FS.
The existing code used RNDD, which rounds down, rather than toward zero.
Kenneth Graunke [Thu, 14 Oct 2010 17:51:09 +0000 (10:51 -0700)]
glsl: Refresh autogenerated file builtin_function.cpp.
Kenneth Graunke [Thu, 14 Oct 2010 17:49:13 +0000 (10:49 -0700)]
glsl: Add front-end support for the "trunc" built-in.
Kenneth Graunke [Wed, 13 Oct 2010 23:35:34 +0000 (16:35 -0700)]
i965: Use logical-not when emitting ir_unop_ceil.
Fixes piglit test glsl-fs-ceil.
Eric Anholt [Thu, 14 Oct 2010 18:27:17 +0000 (11:27 -0700)]
i965: Add peepholing of conditional mod generation from expressions.
This cuts usually 2 out of 3 instructions for flag reg generation (if
statements, conditional assignment) by producing the conditional mod
in the expression representing the boolean value.
Fixes glsl-fs-vec4-indexing-temp-dst-in-nested-loop-combined (register
allocation no longer fails for the conditional generation
proliferation)
Eric Anholt [Thu, 14 Oct 2010 18:11:29 +0000 (11:11 -0700)]
i965: Add a function for handling the move of boolean values to flag regs.
This will be a place to peephole comparisions directly to the flag
regs, and for now avoids using MOV with conditional mod on gen6, which
is now illegal.
Kristian Høgsberg [Thu, 14 Oct 2010 18:57:47 +0000 (14:57 -0400)]
Only install vtxfmt tables for OpenGL
GLES1 and GLES2 install their own exec pointers and don't need the
Save table. Also, the SET_* macros use different indices for the different
APIs so the offsets used in vtxfmt.c are actually wrong for the ES APIs.
Eric Anholt [Thu, 14 Oct 2010 03:17:15 +0000 (20:17 -0700)]
i965: Add a pass to the FS to split virtual GRFs to float channels.
Improves nexuiz performance 0.91% (+/- 0.54%, n=8)
Eric Anholt [Thu, 14 Oct 2010 05:39:06 +0000 (22:39 -0700)]
i965: Update the live interval when coalescing regs.
Eric Anholt [Thu, 14 Oct 2010 02:21:32 +0000 (19:21 -0700)]
i965: Set class_sizes[] for the aligned reg pair class.
So far, I've only seen this be a valgrind warning and not a real failure.
Keith Whitwell [Thu, 14 Oct 2010 12:15:28 +0000 (13:15 +0100)]
llvmpipe: don't try to emit non-existent color outputs
Kristian Høgsberg [Wed, 13 Oct 2010 15:11:47 +0000 (11:11 -0400)]
Drop the "neutral" tnl module
Just always check for FLUSH_UPDATE_CURRENT and call Driver.BeginVertices
when necessary. By using the unlikely() macros, this ends up as
a 10% performance improvement (for isosurf, anyway) over the old,
complicated function pointer swapping.
Chia-I Wu [Thu, 14 Oct 2010 09:13:36 +0000 (17:13 +0800)]
st/egl: Do not finish a fence that is NULL.
i915g would dereference the NULL pointer.
Chia-I Wu [Thu, 14 Oct 2010 09:06:21 +0000 (17:06 +0800)]
st/egl: Access _EGLConfig directly.
Drop the use of SET_CONFIG_ATTRIB. Fix the value of EGL_SAMPLE_BUFFERS
along the way.
Chia-I Wu [Thu, 14 Oct 2010 08:38:58 +0000 (16:38 +0800)]
egl: Access config attributes directly.
Replace SET_CONFIG_ATTRIB/GET_CONFIG_ATTRIB by direct dereferences.
Chia-I Wu [Thu, 14 Oct 2010 08:25:30 +0000 (16:25 +0800)]
egl: Use attribute names as the _EGLConfig member names.
This makes _EGLConfig more accessible and scales better when new
attributes are added.
Dave Airlie [Thu, 14 Oct 2010 04:27:34 +0000 (14:27 +1000)]
r600g: select linear interpolate if tgsi input requests it
Dave Airlie [Thu, 14 Oct 2010 04:19:37 +0000 (14:19 +1000)]
r600g: fixup typo in macro name
Dave Airlie [Thu, 14 Oct 2010 04:14:22 +0000 (14:14 +1000)]
r600g: fixup pos/face ena/address properly
Dave Airlie [Thu, 14 Oct 2010 03:29:11 +0000 (13:29 +1000)]
r600g: only pick centroid coordinate when asked.
TGSI tells us when to use this, its not hooked up from GLSL to MESA to TGSI yet though.
Zhenyu Wang [Thu, 14 Oct 2010 03:24:49 +0000 (11:24 +0800)]
Revert "i965: fallback lineloop on sandybridge for now"
This reverts commit
73dab75b4165f7d2214a68d4ba8e3cb7aab9b4ac.
Zhenyu Wang [Thu, 14 Oct 2010 02:54:53 +0000 (10:54 +0800)]
i965: Fix GS hang on Sandybridge
Don't use r0 for FF_SYNC dest reg on Sandybridge, which would
smash FFID field in GS payload, that cause later URB write fail.
Also not use r0 in any URB write requiring allocate.
Eric Anholt [Wed, 13 Oct 2010 23:13:45 +0000 (16:13 -0700)]
i965: Add support for rescaling GL_TEXTURE_RECTANGLE coords to new FS.
Fredrik Höglund [Wed, 13 Oct 2010 15:49:15 +0000 (17:49 +0200)]
r600g: Fix texture sampling with swizzled coords
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Wed, 13 Oct 2010 23:10:16 +0000 (09:10 +1000)]
r600g: drop unused context members
Ian Romanick [Mon, 11 Oct 2010 23:19:19 +0000 (16:19 -0700)]
mesa: Clean up various 'unused parameter' warnings in shaderapi
Ian Romanick [Mon, 11 Oct 2010 23:13:11 +0000 (16:13 -0700)]
mesa: Clean up two 'comparison between signed and unsigned' warnings
Ian Romanick [Mon, 11 Oct 2010 22:21:17 +0000 (15:21 -0700)]
mesa: Refactor validation of shader targets
Actually validate that the implementation supports the particular
shader target as well. Previously if a driver only supported vertex
shaders, for example, glCreateShaderObjectARB would gladly create a
fragment shader.
NOTE: this is a candidate for the 7.9 branch.
Ian Romanick [Tue, 12 Oct 2010 22:29:43 +0000 (15:29 -0700)]
mesa: Silence unused variable warning
Ian Romanick [Wed, 13 Oct 2010 22:13:02 +0000 (15:13 -0700)]
linker: Reject shaders that have unresolved function calls
This really amounts to just using the return value from
link_function_calls. All the work was being done, but the result was
being ignored.
Fixes piglit test link-unresolved-funciton.
NOTE: this is a candidate for the 7.9 branch.
Vinson Lee [Wed, 13 Oct 2010 21:21:08 +0000 (14:21 -0700)]
glsl: Initialize variable in ir_derefence_array::constant_expression_value
Completely initialize data passed to ir_constant constructor.
Fixes piglit glsl-mat-from-int-ctor-03 valgrind uninitialized value
error on softpipe.
José Fonseca [Wed, 13 Oct 2010 19:25:17 +0000 (20:25 +0100)]
llvmpipe: Generalize the x8z24 fast path to all depth formats.
Together with the previous commit, this generalize the benefits of
d2cf757f44f4ee5554243f3279483a25886d9927 to all depth formats, in
particular:
- simpler float -> 24unorm conversion
- avoid unsigned comparisons (not directly supported on SSE) by aligning
to the least significant bit
- avoid unecessary/repeated mask ANDing
Verified with trivial/tri-z that the exact same assembly is produced for
X8Z24.
José Fonseca [Wed, 13 Oct 2010 14:45:24 +0000 (15:45 +0100)]
gallivm: More accurate float -> 24bit & 32bit unorm conversion.
Brian Paul [Wed, 13 Oct 2010 18:35:38 +0000 (12:35 -0600)]
gallivm: work-around trilinear mipmap filtering regression with LLVM 2.8
The bug only happens on the AOS / fixed-pt path.
Vinson Lee [Wed, 13 Oct 2010 18:18:40 +0000 (11:18 -0700)]
gallivm: Remove unnecessary header.
Brian Paul [Wed, 13 Oct 2010 14:32:08 +0000 (08:32 -0600)]
x11: fix breakage from gl_config::visualType removal
José Fonseca [Wed, 13 Oct 2010 13:28:51 +0000 (14:28 +0100)]
llvmpipe: Unbreak Z32_FLOAT.
Z32_FLOAT uses <4 x float> as intermediate/destination type,
instead of <4 x i32>.
The necessary bitcasts got removed with commit
5b7eb868fde98388d80601d8dea39e679828f42f
Also use depth/stencil type and build contexts consistently, and
make the depth pointer argument a ordinary <i8 *>, to catch this
sort of issues in the future (and also to pave way for Z16 and
Z32_FLOAT_S8_X24 support).
Kristian Høgsberg [Tue, 12 Oct 2010 16:26:10 +0000 (12:26 -0400)]
Drop GLcontext typedef and use struct gl_context instead
Kristian Høgsberg [Tue, 12 Oct 2010 16:02:01 +0000 (12:02 -0400)]
Drop GLframebuffer typedef and just use struct gl_framebuffer
Kristian Høgsberg [Tue, 12 Oct 2010 15:58:47 +0000 (11:58 -0400)]
Rename GLvisual and __GLcontextModes to struct gl_config
Kristian Høgsberg [Tue, 12 Oct 2010 15:40:05 +0000 (11:40 -0400)]
gl: Remove unused GLcontextModes fields
Kristian Høgsberg [Tue, 12 Oct 2010 14:34:03 +0000 (10:34 -0400)]
Get rid of GL/internal/glcore.h
__GLcontextModes is always only used as an implementation internal struct
at this point and we shouldn't install glcore.h anymore. Anything that
needs __GLcontextModes should just include the struct in its headers files
directly.
Roland Scheidegger [Wed, 13 Oct 2010 13:26:37 +0000 (15:26 +0200)]
gallivm: only use lp_build_conv 4x4f -> 1x16 ub fastpath with sse2
This is relying on lp_build_pack2 using the sse2 pack intrinsics which
handle clamping.
(Alternatively could have make it use lp_build_packs2 but it might
not even produce more efficient code than not using the fastpath
in the first place.)
Dave Airlie [Wed, 13 Oct 2010 08:50:37 +0000 (18:50 +1000)]
r600g: fix stencil export for evergreen harder
Stephan Schmid [Mon, 11 Oct 2010 13:49:56 +0000 (15:49 +0200)]
r600g: fix relative addressing when splitting constant accesses
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Wed, 13 Oct 2010 07:45:10 +0000 (17:45 +1000)]
r600g: add missing eg reg definition
Dave Airlie [Wed, 13 Oct 2010 07:40:32 +0000 (17:40 +1000)]
r600g: evergreen add stencil export bit
Dave Airlie [Wed, 13 Oct 2010 01:17:44 +0000 (11:17 +1000)]
r600g: use blitter for hw copy region
at the moment depth copies are failing (piglit depth-level-clamp)
so use the fallback for now until get some time to investigate.
Dave Airlie [Wed, 13 Oct 2010 01:08:44 +0000 (11:08 +1000)]
r600g: drop all use of unsigned long
this changes size on 32/64 bit so is definitely no what you want to use here.
Dave Airlie [Wed, 13 Oct 2010 00:14:55 +0000 (10:14 +1000)]
r600g: fix transfer stride.
fixes segfaults
Dave Airlie [Wed, 13 Oct 2010 01:03:18 +0000 (11:03 +1000)]
r600g: remove bpt and start using pitch_in_bytes/pixels.
this mirror changes in r300g, bpt is kinda useless when it comes to some
of the non-simple texture formats.
Dave Airlie [Wed, 13 Oct 2010 01:02:52 +0000 (11:02 +1000)]
r600g: rename pitch in texture to pitch_in_bytes
Dave Airlie [Tue, 12 Oct 2010 00:51:03 +0000 (10:51 +1000)]
r600g: use common texture object create function
Dave Airlie [Mon, 11 Oct 2010 23:53:17 +0000 (09:53 +1000)]
r600g: split out miptree setup like r300g
just a cleanup step towards tiling
Dave Airlie [Tue, 12 Oct 2010 01:54:16 +0000 (11:54 +1000)]
r600g: add copy into tiled texture
Dave Airlie [Wed, 13 Oct 2010 05:56:12 +0000 (15:56 +1000)]
r600g: the vs/ps const arrays weren't actually being used.
completely removed them.
Dave Airlie [Wed, 13 Oct 2010 05:22:04 +0000 (15:22 +1000)]
r600g: reduce size of context structure.
this thing will be in the cache a lot, so having massive big struct
arrays inside it won't be helping anyone.
Vinson Lee [Wed, 13 Oct 2010 05:15:53 +0000 (22:15 -0700)]
tdfx: Silence unused variable warning on non-debug builds.
Fixes this GCC warning.
tdfx_texman.c: In function 'tdfxTMMoveOutTM_NoLock':
tdfx_texman.c:897: warning: unused variable 'shared'
Dave Airlie [Tue, 12 Oct 2010 03:24:01 +0000 (13:24 +1000)]
r600g: store samplers/views across blit when we need to modify them
also fixup framebuffer state copies to avoid bad state.
Dave Airlie [Wed, 13 Oct 2010 04:23:36 +0000 (14:23 +1000)]
r600g: fix scissor/cliprect confusion
gallium calls them scissors, but r600 hw like r300 is better off using
cliprects to implement them as we can turn them on/off a lot easier.
Dave Airlie [Tue, 12 Oct 2010 04:43:44 +0000 (14:43 +1000)]
r600g: fix depth0 setting
Vinson Lee [Wed, 13 Oct 2010 05:02:27 +0000 (22:02 -0700)]
r300: Silence uninitialized variable warning.
Fixes this GCC warning.
r300_state.c: In function 'r300InvalidateState':
r300_state.c:2247: warning: 'hw_format' may be used uninitialized in this function
r300_state.c:2247: note: 'hw_format' was declared here
Brian Paul [Wed, 13 Oct 2010 00:57:43 +0000 (18:57 -0600)]
mesa: reformatting, comments, code movement
Brian Paul [Wed, 13 Oct 2010 00:38:22 +0000 (18:38 -0600)]
draw/llvmpipe: replace DRAW_MAX_TEXTURE_LEVELS with PIPE_MAX_TEXTURE_LEVELS
There's no apparent reason for the former to exist. And they didn't
even have the same value.
Brian Paul [Wed, 13 Oct 2010 00:37:30 +0000 (18:37 -0600)]
gallivm: remove newlines
Roland Scheidegger [Wed, 13 Oct 2010 00:34:08 +0000 (02:34 +0200)]
gallivm: fix different handling of [non]normalized coords in linear soa path
There seems to be no reason for it, so do same math for both
(except the scale mul, of course).
Brian Paul [Wed, 13 Oct 2010 00:32:06 +0000 (18:32 -0600)]
mesa: remove assertion w/ undeclared variable texelBytes
Dave Airlie [Tue, 5 Oct 2010 23:36:56 +0000 (09:36 +1000)]
st/mesa: enable stencil shader export extension if supported
Dave Airlie [Tue, 5 Oct 2010 23:36:02 +0000 (09:36 +1000)]
glsl: add support for shader stencil export
This adds proper support for the GL_ARB_shader_stencil_export extension
to the GLSL compiler. Thanks to Ian for pointing out where I need to add things.
Dave Airlie [Wed, 6 Oct 2010 00:14:33 +0000 (10:14 +1000)]
r600g: add shader stencil export support.
Dave Airlie [Thu, 7 Oct 2010 03:01:37 +0000 (13:01 +1000)]
r600g: add support for S8, X24S8 and S8X24 sampler formats.
Dave Airlie [Tue, 5 Oct 2010 23:35:07 +0000 (09:35 +1000)]
st/mesa: use shader stencil export to accelerate shader drawpixels.
If the pipe driver has shader stencil export we can accelerate DrawPixels
using it. It tries to pick an S8 texture and works its way to X24S8 and S8X24
if that isn't supported.
Dave Airlie [Thu, 7 Oct 2010 04:04:03 +0000 (14:04 +1000)]
st/mesa: add option to choose a texture format that we won't render to.
We need a texture to put the drawpixels stuff into, an S8 texture is less
memory/bandwidth than the 32-bit X24S8, but we might not be able to render
directly to an S8, so this lets us specify we won't be rendering to this
texture.
Dave Airlie [Tue, 5 Oct 2010 23:32:32 +0000 (09:32 +1000)]
softpipe: add support for shader stencil export capability
this allows softpipe to be used to test shader stencil ref exporting.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Tue, 5 Oct 2010 23:36:15 +0000 (09:36 +1000)]
mesa: improve texstore for 8/24 formats and add texstore for S8.
this improves mesa texstore for 8/24 so it can create S24X8/X24S8 variants
by keeping the depth bits static.
it also adds a texstore for S8 so we can write out an S8 texture to use
in the sampler for accel draw pixels to save memory bw.
The logic seems sound here, I've worked it out a few times on paper, though
it would be good to have some review.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Tue, 5 Oct 2010 23:33:23 +0000 (09:33 +1000)]
mesa: add support for FRAG_RESULT_STENCIL.
this is needed to add support for stencil shader export.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Thu, 7 Oct 2010 05:34:31 +0000 (15:34 +1000)]
gallium/util: add S8 tile sampling support.
Dave Airlie [Thu, 7 Oct 2010 06:14:59 +0000 (16:14 +1000)]
gallium/format: add X32_S8X24_USCALED format.
Has similiar use cases to the S8X24 and X24S8 formats.
Dave Airlie [Tue, 5 Oct 2010 23:30:17 +0000 (09:30 +1000)]
gallium/format: add support for X24S8 and S8X24 formats.
these formats are needed for hw that can sample and write stencil values.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Dave Airlie [Tue, 5 Oct 2010 23:28:46 +0000 (09:28 +1000)]
gallium/tgsi: add support for stencil writes.
this adds the capability + a stencil semantic id, + tgsi scan support.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Eric Anholt [Tue, 12 Oct 2010 21:39:12 +0000 (14:39 -0700)]
i965: Don't rebase the index buffer to min 0 if any arrays are in VBOs.
There was a check to only do the rebase if we didn't have everything
in VBOs, but nexuiz apparently hands us a mix of VBOs and arrays,
resulting in blocking on the GPU to do a rebase.
Improves nexuiz 800x600, high-settings performance on my Ironlake 41%
(+/- 1.3%), from 14.0fps to 19.7fps.
Eric Anholt [Tue, 12 Oct 2010 21:08:00 +0000 (14:08 -0700)]
intel: Allow CopyTexSubImage to InternalFormat 3/4 textures, like RGB/RGBA.
The format selection of the CopyTexSubImage is pretty bogus still, but
this at least avoids software fallbacks in nexuiz, bringing
performance from 7.5fps to 12.8fps on my machine.
Eric Anholt [Tue, 12 Oct 2010 18:36:56 +0000 (11:36 -0700)]
i965: Fix missing "break;" in i2b/f2b, and missing AND of CMP result.
Fixes glsl-fs-i2b.