search

KhronosGroup / SPIRV-Tools

Apache License 2.0
1.08k stars 555 forks source link

spirv-fuzz: Adjust the limit on the maximum number of parameters #3424

Open Vasniktel opened 4 years ago

Vasniktel commented 4 years ago

TransformationAddParameters randomly adds new parameters to the function. However, we must make sure that we don't exceed the universal limit on this quantity which is 255 as of today (section 2.17 is SPIR-V spec). We also wouldn't want to end up having all functions with the 255 parameters. Thus, we are using a custom limit - 128 in FuzzerContext. Consider whether there is a better alternative for this number.

afd commented 4 years ago

@Vasniktel this is a good one to bear in mind when we think about the new DecisionMaker class (or whatever we are going to call it).