llvmpipe: change LP_MAX_SHADER_INSTRUCTIONS limit definition.
authorRoland Scheidegger <sroland@vmware.com>
Thu, 8 May 2014 14:25:47 +0000 (16:25 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Thu, 8 May 2014 14:26:49 +0000 (16:26 +0200)
When the limit was changed to be defined in terms of LP_MAX_SHADER_VARIANTS
(75f1fea14f524ef05e980d825fda3ae226ae2ffe) when it was increased, this
inadvertently lowered the limit in some branches (that have a lower
LP_MAX_SHADER_VARIANTS number) when merged. So, make sure the limit is always
at least the number it once was.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
src/gallium/drivers/llvmpipe/lp_limits.h

index 5ea0c83723adb5a8fefe74cc59fa4ba968e22037..131c02e4bb92ac2bb9d6c7cfa2c76be14da13bfe 100644 (file)
 /**
  * Max number of instructions (for all fragment shaders combined per context)
  * that will be kept around (counted in terms of llvm ir).
+ * Note: the definition looks odd, but there's branches which use a different
+ * number of max shader variants.
  */
-#define LP_MAX_SHADER_INSTRUCTIONS (512*LP_MAX_SHADER_VARIANTS)
+#define LP_MAX_SHADER_INSTRUCTIONS MAX2(128*1024, 512*LP_MAX_SHADER_VARIANTS)
 
 /**
  * Max number of setup variants that will be kept around.