microsoft / D3D12TranslationLayer

A library containing utilities for mapping higher-level graphics work to D3D12
MIT License
327 stars 47 forks source link

Switch to using CopyDescriptorsSimple #58

Closed vdwtanner closed 2 years ago

vdwtanner commented 2 years ago

Pushing this since Alex mentioned that there was actually a (small) measurable performance improvement using CopyDescriptorsSimple afterall.

vdwtanner commented 2 years ago

Alex said that it improved things by about 1FPS, but I'm not sure how he verified that that was statistically significant since that's less than a 1% change. We can hold off on merging this if you want.

jenatali commented 2 years ago

Yeah, it seems technically correct, I'm just not convinced that it's going to be a win across all drivers. Let's hold off for now I think.

vdwtanner commented 2 years ago

We'll be keeping the original implementation, so closing this PR.