On Thursday, March 16, 2017 5:06:55 PM PDT Samuel Pitoiset wrote: > Looks easier to do that at lowering time and mostly because > builtin_builder is a singleton class without access to any > states/constants. > > Signed-off-by: Samuel Pitoiset <[email protected]> > --- > src/compiler/Makefile.sources | 1 + > src/compiler/glsl/ir_optimization.h | 2 + > src/compiler/glsl/lower_sqrt.cpp | 77 > +++++++++++++++++++++++++++++++++++++ > src/compiler/glsl/test_optpass.cpp | 2 + > 4 files changed, 82 insertions(+) > create mode 100644 src/compiler/glsl/lower_sqrt.cpp
Why not put this in lower_instructions.cpp? That's sort of the catch-all pass for tiny bits of expression lowering.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
