vk/vulkan.h: Remove the pass argument from EndRenderPass

vk/vulkan.h: Rename VertexInputStateInfo to VertexInputStateCreateInfo

vk/vulkan.h: Remove programPointSize

Instead, we auto-detect whether or not your shader writes gl_PointSize.  If
it does, we use 1.0, otherwise we take it from the shader.

vk/0.130: Stub vkCmdClear*Attachment() funcs

vkCmdClearColorAttachment
vkCmdClearDepthStencilAttachment

vk/0.130: Define enum VkImageAspectFlagBits

vk/0.130: Define VkRect3D

vk/0.130: Update name of vkCmdClearDepthStencilImage()

vk/0.130: Stub vkCmdExecuteCommands()

vk/0.130: Update vkCmdWaitEvents() signature

vk: Remove unused 'v' param from stub() macro

vk: Stub vkCmdPushConstants() from 0.130 header

vk: Update vkCmdPipelineBarrier to 0.130 header

vk: Add func anv_clear_mask()

A little helper func for inspecting and clearing bitmasks.

vk/vulkan.h: Remove some unused fields.

In particular, the following are removed:

 - disableVertexReuse
 - clipOrigin
 - depthMode
 - pointOrigin
 - provokingVertex

vk/vulkan.h: Remove the explicit primitive restart index

Unfortunately, this requires some non-trivial changes to the driver.  Now
that the primitive restart index isn't given explicitly by the client, we
always use ~0 for everything like D3D does.  Unfortunately, our hardware is
awesome and a 32-bit version of ~0 doesn't match any 16-bit values.  This
means, we have to set it to either UINT16_MAX or UINT32_MAX depending on
the size of the index type.  Since we get the index type from
CmdBindIndexBuffer and the rest of the VF packet from the pipeline, we need
to lazy-emit the VF packet.

vk: Delete some comments not present in 0.130 header

Deleting the comments reduces diff noise.

vk: Pull in remaining 0.130 handle types

This pulls in the definition of VkShaderModule and VkPipelineCache,
which nowhere used yet.

vk: Pull in #defines from 0.130 header

Despite not being used yet, pulling in the macros does diminish the
header diff.

vk/vulkan.h: Rename (min|max)Depth to (min|max)DepthBounds

vk/vulkan.h: Remove client-settable pointSize from DynamicRsState

vk/vulkan.h: Remove UINT8 index buffers

vk/vulkan.h: Re-order the object declarations

vk: Emit 3DSTATE_SAMPLE_MASK

This was missing and was causing the driver to not work with
execlists. Presumably we get a different initial hw context with
execlists enabled, that has sample mask 0 initially.

Set this to 0xffff for now.  When we add MS support, we need to take the
value from VkPipelineMsStateCreateInfo::sampleMask.

vk: Pull in new generated headers

The new headers use stdbool for enable/disable fields which
implicitly converts expressions like (flags & 8) to 0 or 1.
Also handles MBO (must-be-one) fields by setting them to one,
corrects a bspec typo (_3DPRIM_LISTSTRIP_ADJ -> LINESTRIP) and
makes a few enum values less clashy.

vk/image: Validate vkCreateImageView more

Exhaustively validate the function input.  If it's not validated and
doesn't have an anv_finishme(), then I overlooked it.

vk/image: Add more info to VkImageViewType table

Convert the table from the direct mapping
  VkImageViewType -> SurfaceType

into a mapping to an info struct
  VkImageViewType -> struct anv_image_view_info

vk: Update VkImageViewType to 0.130.0

This splits 1D and 1D_ARRAY, 2D and 2D_ARRAY, CUBE and CUBE_ARRAY.

The new tokens are unused. This is just a header update.

vk/image: Move validation for vkCreateImageView

Move the validation from anv_CreateImageView() and anv_image_view_init()
to anv_validate_CreateImageView(). No new validation is added.

vk/vulkan.h: Rename VkRect to VkRect2D

vk/vulkan.h: Rename count to arraySize in VkDescriptorSetLayoutBinding

vk/vulkan.h: Remove the Vk(Memory|Semaphor|Image)OpenInfo structs

We already deleted the functions that need them.  The structs are just
dangling uselessly.

vk/vulkan.h: Remove VK_MEMORY_PROPERTY_PREFER_HOST_LOCAL_BIT

We weren't doing anything with it, so this is a no-op

vk/vulkan.h: Replace DEVICE_COHERENT_BIT with DEVICE_NON_COHERENT_BIT

vk/vulkan.h: Use the query result bits from revision 130

None of the important bits or names actually changed.  It just
added/removed some no-op names.

No functional change.

vk/vulkan.h: One more quick enum refactor clean-up

vk/vulkan.h: Get rid of VERTEX_INPUT_STEP_RATE_DRAW

We never supported it, so no functional change.

vk/vulkan.h: Remove the CLEAR_OPTIMAL image layout

vk: Rename CPU_READ/WRITE_BIT to HOST_READ/WRITE_BIT

vk/vulkan.h: Remove VK_IMAGE_CREATE_CLONEABLE_BIT

vk: Remove SHAREABLE bits

They were removed from the Vulkan API and we don't really use them because
there are no multi-GPU i965 systems.

vk/vulkan.h: Re-order the logic op enums

vk/vulkan.h: Reformat a bunch of enums to match revision 130

In theory, no functional change.

vk: Rename NUM_SHADER_STAGE to SHADER_STAGE_NUM

This is a refactor of more than just the header but it lets us finish
reformating the shader stage enum.

vk: Reformat a bunch of enums

This accounts for a number differences between the generated headers and
the hand-written header.  Not all reformatting is done in this commit but
it does make the headers much more diffable.

In theory, no functional change.

vk: Pull the VkResult enum from revision 130

vk: re-arrange enums to match the order in revision 130

vk: Rename a parameter in CmdBindDynamicStateObject

vk: Remove multi-device stuff

vk: Remove ClearDescriptorSets

vk: Remove begin/end descriptor pool update

vk: Remove stub for CloneImageData

vk: Remove the stub support for memory priorities

vk: Remove the stub support for explicit memory references

vk/vulkan.h: Reformat structs to match revision 130

Structs in the old version were specified as

typedef struct VkSomeThing_
{
   type                                        field; // comment
} VkSomeThing;

However, in the generated headers, you have

typedef struct {
   type                                        field;
} VkSomeThing;

This commit also removes some unneeded whitespaces.

vk/vulkah.h: Re-arrange structures to match the order in 130

nir/spirv: fix some bugs

nir/spirv: add support for builtins inside structures

We may be able to revert this depending on the outcome of bug 14190, but
for now it gets vertex shaders working with SPIR-V.

nir/spirv: fix a bug with structure creation

We were creating 2 extra bogus fields.

nir/spirv: fix a bad assertion in the decoration handling

We should be asserting that the parent decoration didn't hand us
a member if the child decoration did, but different child decorations
may obviously have different members.

nir/spirv: pull out logic for getting builtin locations

Also add support for more builtins.

nir/spirv: plumb through the type of dereferences

We need this to know if a deref is of a builtin.

nir/spirv: handle structure member builtin decorations

nir/spirv: add a vtn_type struct

This will handle decorations that aren't in the glsl_type.

nir/spirv: move 'type' into the union

Since SSA values now have their own types, it's more convenient to make
'type' only used when we want to look up an actual SPIR-V type, since
we're going to change its type soon to support various decorations that
are handled at the SPIR-V -> NIR level.

vk: Move CreateFramebuffer and CreateRenderPass higher in the header

This matches where they are in the 130 header.

vk: Remove atomic counters stubs

vk: Make vulkan.h look more like vulkan-130.h

Most of these changes are insubstantial.  The only potentially substantial
cyhange is that we added a few new #defines for API maximums.

vk: Add a revision 130 header along-side the current header

vk/meta: Add an initial implementation of ClearColorImage

vk/meta: Factor the guts out of cmd_buffer_clear

vk: Roll back to API v90

This is what version 0.1 of the Vulkan SDK is built against.

nir/spirv: Add initial structure member decoration support

nir/spirv: Make vtn_handle_type match the other handler functions

Previously, the caller of vtn_handle_type had to handle actually inserting
the type.  However, this didn't really work if the type was decorated in
any way.

nir/spirv: Add basic support for Op[Group]MemberDecorate

vk/x11: Allow for the client querying the size of the format properties

vk/formats: Fix entry for S8_UINT

I forgot to update this when fixing the depth formats.

vk/formats: Document new meaning of anv_format::cpp

The way the code currently works is that anv_format::cpp is the cpp of
anv_format::surface_format.

Me and Kristian disagree about how the code *should* work. Despite that,
I think it's in our discussion's best interest to document how the code
*currently* works. That should eliminate confusion.

If and when the code begins to work differently, then we'll update the
anv_format comments.

vk/depth: Add a FIXME

3DSTATE_DEPTH_BUFFER.Width,Height are wrong.

vk/image: Enable 2d single-sample color miptrees

What's been tested, for both image views and color attachment views:

    - VK_FORMAT_R8G8B8A8_UNORM
    - VK_IMAGE_VIEW_TYPE_2D
    - mipLevels: 1, 2
    - baseMipLevel: 0, 1
    - arraySize: 1, 2
    - baseArraySlice: 0, 1

What's known to be broken:

    - Depth and stencil miptrees. To fix this, anv_depth_stencil_view
      needs major rework.
    - VkImageViewType != 2D
    - MSAA

Fixes Crucible tests:

  func.miptree.view-2d.levels02.array01.*
  func.miptree.view-2d.levels01.array02.*
  func.miptree.view-2d.levels02.array02.*

vk/image: Define anv_surface, refactor anv_image

This prepares for upcoming miptree support.

anv_surface is a proxy for color surfaces, depth surfaces, and stencil
surfaces.  Embed two instances of anv_surface into anv_image: the
primary surface (color or depth), and an optional stencil surface.

vk/image: Reformat function signatures

Reformat them to match Mesa code-style.

vk/image: Embed VkImageCreateInfo* into anv_image_create_info

All function signatures that matched this pattern,
  old: f(const VkImageCreateInfo *, const struct anv_image_create_info *)

were rewritten as
  new: f(const struct anv_image_create_info *)

vk/image: Drop some tmp vars in anv_image_view_init()

Variables 'tile_mode' and 'format' are unneeded.

vk/image: Abort on stencil image views

The code doesn't work. Not even close.

Replace the broken code with a FINISHME and abort.

vk: Reindent struct anv_image

vk: Define MIN(a, b) macro

vk: Rename functions ALIGN_*32 -> align_*32

ALIGN_U32 and ALIGN_I32 are functions, not macros. So stop using
allcaps.

Merge branch 'wip/nir-vtn' into vulkan

Adds composites and matrix multiplication, plus some control flow fixes.

vk/formats: Remove the cpp=0 stencil hack

The format table defined cpp = 0 for stencil-only formats. The real cpp
is 1.

When code begins to lie, especially about stencil buffers, code becomes
increasingly fragile as time progresses, and the damage becomes
increasingly hard to undo. (For precedent, see the painful history of
stencil buffer cpp in the git log for gen6 and gen7 in the i965 driver).
Let's undo the stencil buffer cpp lie now to avoid future pain.

In the format table, set cpp = 1 for VK_FORMAT_S8; replace checks for
cpp == 0; and delete all comments about the hack.

vk/image: Refactor anv_image_create()

From my experience with intel_mipmap_tree.c, I learned that for struct's
like anv_image and intel_mipmap_tree, which have sprawling
multi-function construction codepaths, it's easy to mistakenly use
unitialized struct members during construction.

Let's eliminate the risk of using unitialized anv_image members during
construction.  Fill the struct at the function bottom instead of
piecemeal throughout the constructor.

vk/image: Group some assertions closer together

In anv_image_create(), group together the assertions on
VkImageCreateInfo.

vk/formats: #undef fmt at end of format table

vk: Fix comment for anv_depth_stencil_view::stencil_qpitch

s/DEPTH/STENCIL/

vk/image: Add qpitch fields to anv_depth_stencil_view

For now, hard-code them to 0.

vk: Reindent and document struct anv_depth_stencil_view

vk/formats: Fix incorrect depth formats

anv_format::surface_format was incorrect for Vulkan depth formats.
For example, the format table mapped

    VK_FORMAT_D24_UNORM -> .surface_format = D24_UNORM_X8_UINT
    VK_FORMAT_D32_FLOAT -> .surface_format = D32_FLOAT

but should have mapped

    VK_FORMAT_D24_UNORM -> .surface_format = R24_UNORM_X8_TYPELESS
    VK_FORMAT_D32_FLOAT -> .surface_format = R32_FLOAT

The Crucible test func.depthstencil.basic passed despite the bug, but
only because it did not attempt to texture from the depth surface.

The core problem is that RENDER_SURFACE_STATE.SurfaceFormat and
3DSTATE_DEPTH_BUFFER.SurfaceFormat are distinct types. Considering them
as enum spaces, the two enum spaces have incompatible collisions.

Fix this by adding a new field 'depth_format' to struct anv_format.

Refer to brw_surface_formats.c:translate_tex_format() for precedent.

vk/image: Rename local variable in anv_image_create()

This function has many local variables for info structs. Having one
named simply 'info' is confusing.  Rename it to 'format_info'.

vk/formats: Fix table entry for R8G8B8_SNORM

Now that anv_formats[] is formatted like a table, buggy entries are
easier to see.

vk/formats: Rename anv_format::format -> surface_format

I misinterpreted anv_format::format as a VkFormat. Instead, it is
a hardware surface format (RENDER_SURFACE_STATE.SurfaceFormat). Rename
the field to 'surface_format' to make it unambiguous.

vk/formats: Rename anv_format::channels -> num_channels

I misinterpreted anv_format::channels as a bitmask of channels.
Renaming it to 'num_channels' makes it unambiguous.