microsoft / microsoft/DirectXShaderCompiler
Support applying clang fix-its automatically
Open
Nobody has claimed this yet.
hlsl-next
- Dominant language
- C++
- Stars
- 3.7k
- Forks
- 900
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 44
Description
Clang supports flags related to fix-its which would be useful to implement in DXC:
-fixit- Applies fix-its to the input file overwriting the file.-fixit=- Applies fix-its and writes out a new file with the specified extension.
We also should consider applying fix-its automatically in the HLSL rewriters as that will extend their utility.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating DXC's command-line handling for Clang's -fixit and -fixit= flags, then trace the HLSL rewriter paths mentioned in the issue. Compare the required behavior with Clang's fix-it support. The work is complete when both file-output modes are supported and automatic application in the HLSL rewriters has been addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100