Hi,
Il 25/01/22 20:06, Zebediah Figura (she/her) ha scritto:
I want to find some way to put "2 dsts" in the function name, because as it is I can't tell just from reading.
Might that be "write_sm4_binary_op_with_two_destinations"?
I'd also suggest specifying both dsts as optional hlsl_ir_node pointers, where NULL translates to SM4_RT_NULL. That would allow you to use this helper even if both dsts are used.
Ok, good idea. Even though in later patches I am always using just one result (and given how our IR is structured, I can't see how it can be different), but generalizing anyway doesn't seem to hurt anybody.
Giovanni.