Re: [Mesa-dev] [PATCH] nv50/ir: swap the least-ref'd source into src1 when both const/imm

2016-01-18 Thread Ilia Mirkin
On Mon, Jan 18, 2016 at 4:14 AM, Ilia Mirkin wrote: > The whole point of inlining sources is to reduce loads. We can end up in > a situation where one value is used a lot of times, and one value is > used only once per instruction. The once-per-instruction one is the one > that should get inlined,

[Mesa-dev] [PATCH] nv50/ir: swap the least-ref'd source into src1 when both const/imm

2016-01-18 Thread Ilia Mirkin
The whole point of inlining sources is to reduce loads. We can end up in a situation where one value is used a lot of times, and one value is used only once per instruction. The once-per-instruction one is the one that should get inlined, but with the previous algorithm, it was given no preference.