On 3/28/18, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> From: Emil Velikov <emil.veli...@collabora.com>
>
> Earlier commit enforced that we'll bail out if the number of terminators
> is different than 2. With that in mind, the assert() will never trigger.
>
> Fixes: 56b867395de ("glsl: fix infinite loop caused by bug in loop
> unrolling pass")
> Cc: Timothy Arceri <tarc...@itsqueeze.com>
> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>Just a nitpick. The explanations doesn't sound right to me. Asserts are meant to never trigger. They are used to check the internal logic of the code. If this assert does trigger that would mean that there is a bug in the code that makes sure the number of terminators is different than 2. It is better to catch bug with assert than to silently do something wrong. Also, sometimes compilers might use the assert assumptions to optimize the code. (Even when the assertion itself is disabled.) Best Regards. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
