Zhengxing li
Zhengxing li
> In your message you say > > > DXC doesn't know (%3, %7) and (%7, %9) are redundant > > How are %7 and %9 redundant here? Should it...
> In your message you say > > > DXC doesn't know (%3, %7) and (%7, %9) are redundant > > How are %7 and %9 redundant here? Should it...
> We should look at the performance impact of this change before committing it. Overall, compared to the upstream change (https://github.com/microsoft/DirectXShaderCompiler/pull/6598), this change can reduce ALU in ~37% of shaders....
> What is the performance impact of this change? I haven't run the perf tests. But I believe the impact's smaller than the one I got before. That's why I...
> > > What is the performance impact of this change? > > > > > > I haven't run the perf tests. But I believe the impact's smaller than...
> I approved the PR because I think this is good, but it would be nice to rebase this on main and put the tests with the other reassociate tests...
I'll work on this task.
I'll work on this task.