Skip to content
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

Update overloading name for all miscellaneous functions. #103

Merged
merged 1 commit into from Jul 14, 2021

Conversation

zakk0610
Copy link
Collaborator

@zakk0610 zakk0610 commented Jul 2, 2021

The one thing should be noted is in the same category function vget and
vset have different overloading naming rule, because vget need to encode
the return type in function name for distinct the different operation.

Copy link
Collaborator

@topperc topperc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The one thing should be noted is in the same category function vget and
vset have different overloading naming rule because vget need to encode
the return type for distinct the different operation.
@zakk0610 zakk0610 merged commit 6ca3259 into master Jul 14, 2021
zakk0610 added a commit to llvm/llvm-project that referenced this pull request Jul 14, 2021
Based on this update to the intrinsic doc
riscv-non-isa/rvv-intrinsic-doc#103

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D105611
arichardson pushed a commit to arichardson/llvm-project that referenced this pull request Sep 13, 2021
Based on this update to the intrinsic doc
riscv-non-isa/rvv-intrinsic-doc#103

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D105611
@zakk0610 zakk0610 deleted the misc_overloading branch February 23, 2022 17:08
mem-frob pushed a commit to draperlaboratory/hope-llvm-project that referenced this pull request Oct 7, 2022
Based on this update to the intrinsic doc
riscv-non-isa/rvv-intrinsic-doc#103

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D105611
vgvassilev pushed a commit to vgvassilev/clang that referenced this pull request Dec 28, 2022
Based on this update to the intrinsic doc
riscv-non-isa/rvv-intrinsic-doc#103

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D105611

llvm-monorepo: 08cf69c31f849310ec45945d18f0feef4ea8f2e6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants