-
Notifications
You must be signed in to change notification settings - Fork 11.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NVPTX] support immediate values in st.param instructions #91523
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious -- do we expect to see any benefits from this in SASS?
ptxas usally deals with mov const->register; op(register)
just fine and with parameters living mostly in registers, the notion of vectorized loads/stores in parameter space is largely a fiction.
What does this change buy us, other than a few less register moves on PTX level?
!strconcat("st.param.v2", opstr, | ||
" \t[param$a+$b], {{$val, $val2}};"), | ||
[]>; | ||
multiclass StoreParamInst<NVPTXRegClass regclass, Operand IMMType, string opstr, bit support_imm = true> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice. I like it even better.
d1fb3df
to
e328d6a
Compare
Add support for generating
st.param
instructions with direct use of immediates. This eliminates the need for amov
instruction prior to thest.param
resulting in more concise emitted PTX.