Repository navigation
impl part of Feature: Native Triton GPU Kernel Strict Typing Plugin #3884 - #5309
asukaminato0721 wants to merge 3 commits into
Conversation
|
This pull request has been imported. If you are a Meta employee, you can view this in D126647218. (Because this pull request was imported automatically, there will not be any future comments.) |
This comment has been minimized.
This comment has been minimized.
|
I'll take a look at this, thanks! I suspect we'll wind up treating this more as a prototype than something to merge quickly, because triton support is a big undertaking and we probably want to tie it to tensor shape types (which also will mean designing an extensive stub library). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅ |
|
Sounds like this is more of a prototype/waiting for interactions with tensor shapes to be worked out, so I'm going to move it to draft so I don't keep seeing it and thinking it needs review :) |
Summary
part of #3884, Pyrefly now treats
tl.constexprparameters as Any in recognized Triton JIT functions.Test Plan
add test