Skip to content

Commit cbf6d48

Browse files
committed
Address review points
1 parent 879c07b commit cbf6d48

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

proposals/NNNN-debug-printf.md

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,13 @@ will be a variable arguments function.
6565

6666
## Detailed design
6767

68-
1. The printf dxil op will be non-semantic, it does not affect final hlsl code/algorithm.
69-
Non-semantic dxil op code can be counted down from 0xffff, or 0xffffffff, it will give a hint to the client api
70-
to remove the non semantic dxil safely
68+
1. The printf dxil op will be purely for debug purpose, it does not affect final hlsl
69+
code/algorithm in any way.
70+
To separate this kind of debug-purpose dxil ops to the normal non-debug dxil il ops,
71+
the debug printf dxil op code can be counted down from 0xffff, or 0xffffffff,
72+
e.g. op code is 0xfffe.
73+
So it will give a hint to the d3d debug layer to pick up debug printf dxil easily,
74+
or a underlying d3d driver to remove these dxil op codes safely.
7175
2. Add a option to enable printf dxil op generation to dxc, try to separate hlsl code for debugging
7276
and for production, if printf option is disabled, the printf in hlsl will be report a error
7377
3. We should not support dynamic string variable, a string variable content.

0 commit comments

Comments
 (0)