[BUG][Inductor-EVT] Python EVT tracer generates incorrect code when assigning accumulator to output D
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 10.5k
- Forks
- 2.1k
- Avg merge
- 3d 11h
- Merged PRs (30d)
- 7
Description
Describe the bug
When assigning accum arg to D, invalid code is generated. Namely, an alias EVTD is used but not defined (line 18 in the output code). This restriction should probably either be a hard error or generate the definition of the type alias.
Steps/Code to reproduce bug
Expected behavior
Generate buildable code
Environment details (please complete the following information):
Meta devgpu, although this should repro on any other machine.
To workaround this I ensure that accum is not assigned to D. (swapping D and E in the example yields valid code)
cc @thakkarV, @mnicely, @henrylhtsang, @eellison
Contributor guide
No contributing guide indexed for this repository
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
Run the linked reproduction script and inspect the linked output code, focusing on the undefined EVTD alias when the accumulator is assigned to D. Trace the Python EVT tracer's generated output and verify that the resulting code builds successfully without an undefined alias.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, python
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100