[SPIR-V][vk::SampledTexture] #5. Add remaining .Sample...() methods for vk::SampledTexture2D` type.

[luciechoi]

Part of #7979

Implement remaining sample methods methods for vk::SampledTexture2D:

float Object.SampleCmp(float Location, float CompareValue, [, int Offset], [, float Clamp] [, out uint Status]);
float Object.SampleCmpLevelZero(float Location, float CompareValue, [, int Offset], [, out uint Status]);
float Object.SampleCmpLevel(float Location, float CompareValue, float LOD, [, int Offset], [, out uint Status]);
float Object.SampleGrad(float Location, float DDX, float DDY, [, int Offset], [, float Clamp] [, out uint Status]);
float Object.SampleCompGrad(float Location, float CompareValue, float DDX, float DDY, [, int Offset], [, float Clamp] [, out uint Status]);
float Object.SampleCompBias(float Location, float CompareValue, float Bias, [, int Offset], [, float Clamp] [, out uint Status]);
PARAM_TYPE Object.SampleBias(float Location, float Bias, [, int Offset], [, float Clamp], [, out uint Status]);
PARAM_TYPE Object.SampleLevel(float Location, float LOD, [, int Offset], [, out uint Status]);

[s-perron]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Keenuts]

I find it confusing to load the clamp value from the offsetIndex. Why is that and not the clampIndex?

[luciechoi]

Thanks for pointing out, I agree it's confusing. There was a discussion around deriving the index from non-optional argument indices. But offset here is also optional, so it's incorrect.

In fact, I noticed there was a helper method to handle these common cases, so refactored. PTAL!