diff options
| author | Matt Turner <mattst88@gmail.com> | 2015-11-20 21:18:26 -0800 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2015-11-24 09:58:33 -0800 |
| commit | f093c842e65b251e24ea3a2d6daaa91326a4f862 (patch) | |
| tree | a6537178d047859337223c0d699f6a1f0f81e27e /src/mesa/drivers/dri/i965/brw_shader.h | |
| parent | 309a44d63c75a7d688157486b094e555f49c907d (diff) | |
| download | external_mesa3d-f093c842e65b251e24ea3a2d6daaa91326a4f862.zip external_mesa3d-f093c842e65b251e24ea3a2d6daaa91326a4f862.tar.gz external_mesa3d-f093c842e65b251e24ea3a2d6daaa91326a4f862.tar.bz2 | |
i965: Use implicit backend_reg copy-constructor.
In order to do this, we have to change the signature of the
backend_reg(brw_reg) constructor to take a reference to a brw_reg in
order to avoid unresolvable ambiguity about which constructor is
actually being called in the other modifications in this patch.
As far as I understand it, the rule in C++ is that if multiple
constructors are available for parent classes, the one closest to you in
the class heirarchy is closen, but if one of them didn't take a
reference, that screws things up.
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_shader.h')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_shader.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_shader.h b/src/mesa/drivers/dri/i965/brw_shader.h index 718a225..abf04cf 100644 --- a/src/mesa/drivers/dri/i965/brw_shader.h +++ b/src/mesa/drivers/dri/i965/brw_shader.h @@ -42,7 +42,7 @@ struct backend_reg : public brw_reg { backend_reg() {} - backend_reg(struct brw_reg reg) : brw_reg(reg) {} + backend_reg(const struct brw_reg ®) : brw_reg(reg) {} bool equals(const backend_reg &r) const; |