Skip to content

Support vector front/back - #702

Merged
anthony-santana merged 6 commits into
NVIDIA:mainfrom
anthony-santana:ast_vector
Oct 3, 2023
Merged

Support vector front/back#702
anthony-santana merged 6 commits into
NVIDIA:mainfrom
anthony-santana:ast_vector

Conversation

@anthony-santana

@anthony-santana anthony-santana commented Sep 27, 2023

Copy link
Copy Markdown
Collaborator

Description

Closes #645

@copy-pr-bot

copy-pr-bot Bot commented Sep 27, 2023

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

schweitzpgi
schweitzpgi reviewed Sep 28, 2023
View reviewed changes

@schweitzpgi schweitzpgi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You do not want to put a Load here. These functions actually return references which are memory addresses rather than the value at the memory address. Internal to the compiler that means effectively there is no difference between

  T& foo();
  T* foo();

in terms of implementation. Both return a memory address. The first does have implicit dereferencing, but that is syntactic sugar that doesn't affect the implementation here.

Comment thread lib/Frontend/nvqpp/ConvertExpr.cpp Outdated
Comment thread lib/Frontend/nvqpp/ConvertExpr.cpp Outdated
@anthony-santana
anthony-santana force-pushed the ast_vector branch 5 times, most recently from f3d1621 to 469e547 Compare October 2, 2023 18:53
@anthony-santana
anthony-santana marked this pull request as ready for review October 2, 2023 19:38
@anthony-santana

anthony-santana commented Oct 2, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator Author

/ok to test

Command Bot: Processing...

@anthony-santana

anthony-santana commented Oct 2, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator Author

/ok to test

Command Bot: Processing...

@github-actions

github-actions Bot commented Oct 2, 2023

Copy link
Copy Markdown

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions Bot pushed a commit that referenced this pull request Oct 2, 2023
@cuda-quantum-bot
Docs preview for PR #702.
0f74792
schweitzpgi
schweitzpgi reviewed Oct 3, 2023
View reviewed changes
Comment thread test/AST-Quake/vector_front_back.cpp Outdated
schweitzpgi
schweitzpgi reviewed Oct 3, 2023
View reviewed changes
Comment thread test/AST-Quake/vector_front_back.cpp Outdated
schweitzpgi
schweitzpgi approved these changes Oct 3, 2023
View reviewed changes

@schweitzpgi schweitzpgi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thanks, Anthony.

@anthony-santana

anthony-santana commented Oct 3, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator Author

/ok to test

Command Bot: Processing...

@github-actions

github-actions Bot commented Oct 3, 2023

Copy link
Copy Markdown

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions Bot pushed a commit that referenced this pull request Oct 3, 2023
@cuda-quantum-bot
Docs preview for PR #702.
3582133
@anthony-santana
anthony-santana enabled auto-merge (squash) October 3, 2023 18:36
@anthony-santana

anthony-santana commented Oct 3, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator Author

/ok to test

Command Bot: Processing...

@github-actions

github-actions Bot commented Oct 3, 2023

Copy link
Copy Markdown

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions Bot pushed a commit that referenced this pull request Oct 3, 2023
@cuda-quantum-bot
Docs preview for PR #702.
8f8fdba
@anthony-santana
initial implementation
611dcd9 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana
remove assert
57b83f7 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana
woring front/back
0efc816 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana
pr comments and proper filecheck test for front back
bb10438 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana
clang format
63be041 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana
check dags
23fda3f 
Signed-off-by: A.M. Santana <anthonys@nvidia.com>
@anthony-santana

anthony-santana commented Oct 3, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator Author

/ok to test

Command Bot: Processing...

@anthony-santana
anthony-santana merged commit c69729b into NVIDIA:main Oct 3, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 3, 2023
@anthony-santana
anthony-santana deleted the ast_vector branch October 3, 2023 23:09
@bettinaheim bettinaheim added this to the release 0.5.0 milestone Nov 15, 2023
@bettinaheim bettinaheim changed the title Support vector front/back in bridge Support vector front/back Nov 15, 2023
@bettinaheim bettinaheim added the release notes Changes need to be captured in the release notes label Nov 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@schweitzpgi schweitzpgi schweitzpgi approved these changes

Assignees

No one assigned

Labels

release notes Changes need to be captured in the release notes

Projects

None yet

Milestone

release 0.5.0

Development

Successfully merging this pull request may close these issues.

Support lowering of std::vector::front and std::vector::back

3 participants

@anthony-santana @schweitzpgi @bettinaheim