diff options
| author | Ian Romanick <[email protected]> | 2011-12-23 10:59:38 -0800 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2012-01-06 14:32:50 -0800 |
| commit | 1f125374e73d1718cd54c946826c76b09998c055 (patch) | |
| tree | eec2e3222c833f2f71bb2cf4a1955fe05194626c /src/glsl/ast_to_hir.cpp | |
| parent | 208b5b113faab324854d457c3ad4abf05cafedd1 (diff) | |
glsl: Don't mark assignment temporaries as read-only
The various l-value errors this was designed to catch are now caught
by other means. Marking the temporaries as read-only now just
prevents sensible error messages from being generated. It's
0:0(0): error: function parameter 'out p' references the read-only variable '_post_incdec_tmp'
versus
0:13(5): error: function parameter 'out p' references a post-decrement operation
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Paul Berry <[email protected]>
Diffstat (limited to 'src/glsl/ast_to_hir.cpp')
| -rw-r--r-- | src/glsl/ast_to_hir.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp index cb2ef429f70..1aebca40fac 100644 --- a/src/glsl/ast_to_hir.cpp +++ b/src/glsl/ast_to_hir.cpp @@ -775,11 +775,6 @@ get_lvalue_copy(exec_list *instructions, ir_rvalue *lvalue) instructions->push_tail(new(ctx) ir_assignment(new(ctx) ir_dereference_variable(var), lvalue, NULL)); - /* Once we've created this temporary, mark it read only so it's no - * longer considered an lvalue. - */ - var->read_only = true; - return new(ctx) ir_dereference_variable(var); } |