On Mon Sep 11 21:50:07 2023 +0000, Zebediah Figura wrote:
I think you misunderstood. What I mean is that explict 'if (x) a=b;
else a=c;' might be turned to conditional move with x.xxxx mask, as an optimization. The helper is internal, but I'd rather remove it when it's not useful. Arguably it's still useful as a convenient shorthand for ternary hlsl_new_expr().
Sure, we can do that. Does it have to be in this MR, or separately? I think it works either way, I wasn't planning on it, but I can amend with int handling changes, together with corresponding tests.