kc3-lang/angle/src/compiler/translator/ValidateMaxParameters.cpp

• Show log

  Commit

• Hash : d4bd963f
  Author :
  Date : 2018-03-08T16:32:44
  

  Don't use TIntermSymbol nodes for function parameters
  
  Parameter nodes are not needed - it's simpler to just create a
  TVariable object for each parameter when the TFunction is initialized.
  
  With this change we also store only one object per each parameter type
  used in built-in functions, instead of one array of TConstParameter
  entries for each unique parameter sequence.
  
  This simplifies code and reduces binary size and compiler memory use.
  Compiler perf does not seem to be significantly affected.
  
  BUG=angleproject:2267
  TEST=angle_unittests
  
  Change-Id: I2b82400dd594731074309f92a705e75135a4c82c
  Reviewed-on: https://chromium-review.googlesource.com/955589
  Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
  Reviewed-by: Jamie Madill <jmadill@chromium.org>
  

Download

• src/compiler/translator/ValidateMaxParameters.cpp

• //
  // Copyright (c) 2016 The ANGLE Project Authors. All rights reserved.
  // Use of this source code is governed by a BSD-style license that can be
  // found in the LICENSE file.
  //
  // ValidateMaxParameters checks if function definitions have more than a set number of parameters.
  
  #include "compiler/translator/ValidateMaxParameters.h"
  
  #include "compiler/translator/IntermNode.h"
  #include "compiler/translator/Symbol.h"
  
  namespace sh
  {
  
  bool ValidateMaxParameters(TIntermBlock *root, unsigned int maxParameters)
  {
      for (TIntermNode *node : *root->getSequence())
      {
          TIntermFunctionDefinition *definition = node->getAsFunctionDefinition();
          if (definition != nullptr &&
              definition->getFunctionPrototype()->getFunction()->getParamCount() > maxParameters)
          {
              return false;
          }
      }
      return true;
  }
  
  }  // namespace sh
  

Download