11:45 <alyssa> bbrezillon: I think you might be the only interested reviewer for https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23836

11:45 <alyssa> I might've gotten carried away..

11:46 <alyssa> I'm not thrilled about the load_frag_coord_zw intrinsic

11:46 <alyssa> Maybe I just want to have the backend replace load_frag_coord.xy with undefs (since they'd be unused) and trust the backend to copy propagate + dead code eliminate them

11:47 <alyssa> That requires a fair degree of backend smarts. but both Bifrost and AGX backends can do it without breaking a sweat

11:47 <alyssa> (Midgard otoh.)

11:55 <alyssa> italove7: For !23769, let me know if you want to jump on a video call or something, I am realizing now that it is certifiably A Lot if you haven't worked on that type of code before

12:12 <italove> alyssa: I've started reviewing it last week, just didn't get to finish it yet. But thanks, if I need it I'll hit you up :)

12:12 <alyssa> :+1:

12:38 <alyssa> OTOH, for AGX it's very convenient to have load_frag_coord_zw scalarized since we allocate different "things" if gl_FragCoord.z is read versus gl_FragCoord.w

12:38 <alyssa> (not super important what "things" are in this context. currently we use nir_ssa_def_components_read for the purpose.)

13:04 <bbrezillon> alyssa: no objection on my side

13:05 <alyssa> :+1: