Skip to content

A
angle
Switch branch/tag
  1. 10 May, 2017 3 commits
  3. 09 May, 2017 1 commit
    • GLSL: Simplify constructor parsing · a7ecec38
      Olli Etuaho authored 
      Constructor argument checking rules are reorganized to make them
easier to understand and constructor node creation is made simpler.
This removes usage of constructor op codes from ParseContext. This
paves the way for getting rid of constructor op codes entirely, which
will remove duplicate information from the AST and simplify lots of
code.

This refactoring will make adding arrays of arrays slightly easier.

BUG=angleproject:1490
TEST=angle_unittests

Change-Id: I4053afec55111b629353b4ff7cb0451c1ae3511c
Reviewed-on: https://chromium-review.googlesource.com/498767
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org>
  5. 08 May, 2017 6 commits
  7. 05 May, 2017 3 commits
  9. 04 May, 2017 5 commits
    • GLES2: Auto-generate entry points file. · ee769dd2
      Jamie Madill authored 
      BUG=angleproject:1309

Change-Id: I7817444c3ea56f932fe769a860f4a70b29262019
Reviewed-on: https://chromium-review.googlesource.com/483427Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org>
Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
    • Inherit privately from angle::NonCopyable. · aa7203ef
      Frank Henigman authored 
      Make all inheritance from angle::NonCopyable private so the compiler
complains about this (admittedly unlikely) code:

  class Foo: angle::NonCopyable {
      virtual ~Foo() { ... }
  };
  angle::NonCopyable *p = new Foo;
  delete p;

In the above code ~Foo() is not called, only ~NonCopyable(), because the
latter is not virtual.  Making it virtual would add overhead to all derived
classes which don't already have a virtual method.

Also tighten access in NonCopyable, because we can.

BUG=angleproject:2026

Change-Id: Id0dc4d959cfb7bb82cf49382118129abb1d3a4f0
Reviewed-on: https://chromium-review.googlesource.com/495352Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org>
Commit-Queue: Frank Henigman <fjhenigman@chromium.org>
    • Fix compilation of the Vulkan loader / layers in standalone · 6a1d2f93
      Corentin Wallez authored 
      This commit introduces a stamping mechanism to make sure the order of
build dependencies is correct.

In GYP there are the following build order dependencies (between others):
 - A target link depends on previous target links
 - A target source compile depends on that target actions outputs
 - An action output depends on the action inputs

The problem was that there were actions in vulkan_layer_utils_static
that generated headers needed by vulkan_loader's sources (between others)
Making a "dependencies" relationship between the two was not enough to
ensure the headers were generated before the source were compiled.

Instead we replicated the "stamping" mechanism used in GN using GYP
actions that depend on the headers and output a dummy stamp file. This
completes the build order dependencies: the sources depend on the stamp
which depend on the action's input, i.e. the generated headers.

BUG=angleproject:1668

Change-Id: I2de15ac2ea8f838acdbf43da1f111b28b8cc2184
Reviewed-on: https://chromium-review.googlesource.com/493892
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: 's avatarFrank Henigman <fjhenigman@chromium.org>
    • Update checks for floating point renderability. · 677bb6ff
      Geoff Lang authored 
       * Expose GL_CHROMIUM_color_buffer_float_rgb and
   GL_CHROMIUM_color_buffer_float_rgba
 * Fix many texture formats that were incorrectly checking the wrong
   extension for support or renderability.
 * Make all floating point texture extensions dynamically enableable.

BUG=angleproject:1958
BUG=angleproject:1715

Change-Id: Iefccc8b5ae5edd97623affa9de05b1d9af5c9598
Reviewed-on: https://chromium-review.googlesource.com/468450Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>
    • GLSL parser: Fix empty declaration qualifier checks · bb7e5a7c
      Olli Etuaho authored 
      The shader validation now does the same checks for qualifier
combinations regardless of if a declaration is empty, as is specified.
Some of these checks used to be in singleDeclarationErrorCheck, and
have now been moved to a new function declarationQualifierErrorCheck.
The other parts of singleDeclarationErrorCheck are under a renamed
nonEmptyDeclarationErrorCheck.

The patch also contains another related cleanup: Unnecessary symbol
nodes won't be created for empty declarations any more.

BUG=angleproject:2020
TEST=angle_unittests

Change-Id: I1c864a5e151c52703926d8c550450b2561bfcbb2
Reviewed-on: https://chromium-review.googlesource.com/493227
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
  11. 03 May, 2017 5 commits
  13. 02 May, 2017 6 commits
  15. 01 May, 2017 2 commits
  17. 28 Apr, 2017 5 commits
  19. 27 Apr, 2017 4 commits