-
Notifications
You must be signed in to change notification settings - Fork 87
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
rdf: Include cstdint
header explicitly for std::int64_t
and friends
#94
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.
Thanks!
27840c7
to
935599d
Compare
935599d
to
3252a38
Compare
On gcc-13 these integer types appear to no longer be included via `(c)string` or other `std` headers (according to a [similar fix]), resulting in many of the following compiler errors in `amdrdf.{cpp,h}`: In file included from pal/src/core/imported/rdf/rdf/src/amdrdf.cpp:25: pal/src/core/imported/rdf/rdf/inc/amdrdf.h:112:39: error: ‘int64_t’ in namespace ‘std’ does not name a type 112 | int (*Read)(void* ctx, const std::int64_t count, void* buffer, std::int64_t* bytesRead); | ^~~~~~~ Solve this by including `cstdint` explicitly. [similar fix]: google/breakpad@7ea7ded
3252a38
to
56868b7
Compare
retest this please |
@jinjianrong what needs to be retested? This repository hasn't changed since this PR was last updated, and it is still required to solve the build error. |
@MarijnS95 This is to tell our internal jenkins to run pre-submission build and tests for the PR, not to you. Sorry for the confusion. |
Can one of the admins verify this patch? |
Test summary for commit 56868b7CTS tests (Failed: 0/213062)
Rhel 9.0, Gfx10Ubuntu 22.04, Navi3xUbuntu 20.04, Navi2x |
1 similar comment
Test summary for commit 56868b7CTS tests (Failed: 0/213062)
Rhel 9.0, Gfx10Ubuntu 22.04, Navi3xUbuntu 20.04, Navi2x |
Ah that is cool, thank you for explaining @jinjianrong! (Perhaps if the wording includes |
This is fixed internally, but it will take a while to trickle through the release branches. I’ll see if we can get this into one of the next releases. |
On gcc-13 these integer types appear to no longer be included via
(c)string
or otherstd
headers (according to a similar fix), resulting in many of the following compiler errors inamdrdf.{cpp,h}
:Solve this by including
cstdint
explicitly.