Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[1/2] Intel GPU Runtime Upstreaming for Generator #118528
[1/2] Intel GPU Runtime Upstreaming for Generator #118528
Changes from 51 commits
8115ce3
1e7a78a
b79871d
cfa8d20
6e04041
f326cac
015afbc
b124234
b08c032
a86685f
3436cab
a9eef30
97d46c1
9cfb90f
6a980a6
01195cc
dde8977
ffcc027
9cf585d
3fd1741
4d3e231
1c22b4b
cbf0fd0
eb7ce1b
adc16bd
9244bd9
92c7960
1a22017
394810a
8425282
69be3b0
e9918aa
7d166f9
3ec9249
164479b
94d76f2
a718e2a
0d01458
2dde0b8
1f4b17a
4c96b2b
ef37ac5
3c0b395
2e463d3
0e2d656
a1cf537
b100940
feb8c86
10cc032
b7b1a81
f12e8ad
c847086
b5ddd52
b6d21d9
859f118
e9cef1a
c015f4a
ae25166
89790a2
954b19c
9cf7d35
2a20a29
75f6754
b1c711b
03217cd
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Why not a vector like below?
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.
change to
std::vector<c10::once_flag> xpu_gens_init_flag;
Here
std::vector
is more efficient thanstd::deque
.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.
Sorry @albanD, looks like
std::vector<c10::once_flag>
doesn't supportresize
method becauseonce_flag
lacks a copy constructor. seepytorch/c10/util/CallOnce.h
Line 37 in 685d862
I have also tried on godbolt. So, here I change back to
std::deque
.I think I missed this error before because I didn't save my code change when I rebuilt it in my local machine. I'm very sorry about this.
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.
@albanD Could you help review again?
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.
Ho interesting. Sounds good!