Acked-by: Jose Maria Casanova Crespo <jmcasan...@igalia.com> Matt, I'll include in my TODO list creating tests for sends using grf127 as destination restriction and the one about byte_raw_moves and execution size and stride once I receive your feedback.
Chema El 24/1/19 a las 20:53, Matt Turner escribió: > --- > src/intel/compiler/brw_eu_validate.c | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/src/intel/compiler/brw_eu_validate.c > b/src/intel/compiler/brw_eu_validate.c > index a25010b225c..7f1580a5bb3 100644 > --- a/src/intel/compiler/brw_eu_validate.c > +++ b/src/intel/compiler/brw_eu_validate.c > @@ -1,5 +1,5 @@ > /* > - * Copyright © 2015 Intel Corporation > + * Copyright © 2015-2019 Intel Corporation > * > * Permission is hereby granted, free of charge, to any person obtaining a > * copy of this software and associated documentation files (the "Software"), > @@ -24,6 +24,18 @@ > /** @file brw_eu_validate.c > * > * This file implements a pass that validates shader assembly. > + * > + * The restrictions implemented herein are intended to verify that > instructions > + * in shader assembly do not violate restrictions documented in the graphics > + * programming reference manuals. > + * > + * The restrictions are difficult for humans to quickly verify due to their > + * complexity and abundance. > + * > + * It is critical that this code is thoroughly unit tested because false > + * results it will lead developers astray, which is worse than having no > + * validator at all. Patches to this file without corresponding unit tests > (in > + * test_eu_validate.cpp) will be rejected. > */ > > #include "brw_eu.h" > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev