DirectXShaderCompiler icon indicating copy to clipboard operation
DirectXShaderCompiler copied to clipboard

Strip struct, entry and groupshared names from DXIL

Open GlassBeaver opened this issue 3 months ago • 6 comments

Strip struct, entry and groupshared names from DXIL when stripping reflection data. https://github.com/microsoft/DirectXShaderCompiler/issues/7627

GlassBeaver avatar Nov 02 '25 15:11 GlassBeaver

:white_check_mark: With the latest revision this PR passed the C/C++ code formatter.

github-actions[bot] avatar Nov 02 '25 15:11 github-actions[bot]

@microsoft-github-policy-service agree

GlassBeaver avatar Nov 02 '25 15:11 GlassBeaver

@llvm-beanz Updated based on the review. Let me know if this is what you meant by using a separate pass. If not, please provide some guidance as to where and how it should work.

GlassBeaver avatar Nov 03 '25 20:11 GlassBeaver

Refactored the pass to roughly where advised and the functionality seems to still work and my tests pass, however CompilerTest::CompileThenTestPdbUtils is now failing. Since you've mentioned wanting to take a look at the tests later this week, I've left it in a failing state since I'm not sure what the interplay is between the passes and those tests with regards to data stripping, i.e. it could be that the failing test needs updating but also that my changes are wrong.

GlassBeaver avatar Nov 04 '25 16:11 GlassBeaver

Synced up to main. @llvm-beanz let me know if you need anything from me regarding those tests

GlassBeaver avatar Nov 11 '25 14:11 GlassBeaver

@llvm-beanz - have you had a chance to double check your feedback has been addressed?

@GlassBeaver - could you add something to ReleaseNotes.md documenting this change please?

damyanp avatar Dec 01 '25 19:12 damyanp