Incorporated patch from haixia@ changing the default implementation of (issue 7194051)

[k...@chromium.org]

Reviewers: apatrick1,

Description:
Incorporated patch from haixia@ changing the default implementation of
array index clamping to use the clamp intrinsic. This works more
reliably on pure OpenGL ES devices and on Windows.

Added a mechanism in ShBuiltInResources to choose the strategy for
array index clamping.

BUG=none
TEST=various out-of-bounds array indexing tests and various WebGL
content

Please review this at https://codereview.appspot.com/7194051/

Affected files:
M include/GLSLANG/ShaderLang.h
M src/common/version.h
M src/compiler/Compiler.cpp
M src/compiler/OutputESSL.cpp
M src/compiler/OutputESSL.h
M src/compiler/OutputGLSL.cpp
M src/compiler/OutputGLSL.h
M src/compiler/OutputGLSLBase.cpp
M src/compiler/OutputGLSLBase.h
M src/compiler/ShHandle.h
M src/compiler/ShaderLang.cpp
M src/compiler/TranslatorESSL.cpp
M src/compiler/TranslatorGLSL.cpp
M src/third_party/compiler/ArrayBoundsClamper.cpp
M src/third_party/compiler/ArrayBoundsClamper.h

[apat...@chromium.org]

LGTM

https://codereview.appspot.com/7194051/

[dan...@transgaming.com]

Is a CLA required for this (has that been covered)?

https://codereview.appspot.com/7194051/

[k...@chromium.org]

No, hshi@ is a Google employee. (Got his address wrong in the CL
description, sorry.)


https://codereview.appspot.com/7194051/

[dan...@transgaming.com]

Ok great. Wasn't sure if he was or not. Thanks!

https://codereview.appspot.com/7194051/