kc3-lang/angle/src/compiler/translator/RemoveNoOpCasesFromEndOfSwitchStatements.h

• Show log

  Commit

• Hash : 923ecef6
  Author :
  Date : 2017-10-11T12:01:38
  

  Fix switch statement validation corner cases
  
  The grammar needs to generate AST nodes even for no-op statements,
  since they might be the last statement in a switch statement that is
  required for switch statement validity. Change the grammar to generate
  nodes from empty blocks and empty declarations.
  
  We also need to do some further processing of the AST. This is because
  PruneEmptyDeclarations will still remove empty declarations, and at
  least the NVIDIA driver GLSL compiler doesn't accept some types of
  no-op statements as the last statement inside a switch statement. So
  after parsing has finished we do rudimentary dead code elimination to
  remove dead cases from the end of switch statements.
  
  BUG=angleproject:2181
  TEST=angle_unittests
  
  Change-Id: I586f2e4a3ac2171e65f1f0ccb7a7de220e3cc225
  Reviewed-on: https://chromium-review.googlesource.com/712574
  Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
  Reviewed-by: Corentin Wallez <cwallez@chromium.org>
  

Download

• src/compiler/translator/RemoveNoOpCasesFromEndOfSwitchStatements.h

• //
  // Copyright (c) 2017 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.
  //
  // RemoveNoOpCasesFromEndOfSwitchStatements.h: Clean up cases from the end of a switch statement
  // that only contain no-ops.
  
  #ifndef COMPILER_TRANSLATOR_REMOVENOOPCASESFROMENDOFSWITCHSTATEMENTS_H_
  #define COMPILER_TRANSLATOR_REMOVENOOPCASESFROMENDOFSWITCHSTATEMENTS_H_
  
  namespace sh
  {
  class TIntermBlock;
  class TSymbolTable;
  
  void RemoveNoOpCasesFromEndOfSwitchStatements(TIntermBlock *root, TSymbolTable *symbolTable);
  
  }  // namespace sh
  
  #endif  // COMPILER_TRANSLATOR_REMOVENOOPCASESFROMENDOFSWITCHSTATEMENTS_H_
  

Download