Skip to content

Conversation

@dbermond
Copy link
Contributor

Needed for building with GCC 15 and later. Otherwise, a build error happens.

This seems to affect only the unit tests of the legacy version, as unit tests of the non-legacy version can be built fine without this change.

Same situation of commit: e0362a7

See the mentioned commit message for the full explanation.

@JablonskiMateusz
Copy link
Contributor

@dbermond please correct commit msg according to https://github.com/intel/compute-runtime/blob/master/CONTRIBUTING.md

  • missing signed-off-by
  • each line has 80 character limit

Needed for building with GCC 15 and later. Otherwise, a build
error happens.

Same situation of commit e0362a7.

See the mentioned commit message for the full explanation.

Signed-off-by: Daniel Bermond <[email protected]>
@dbermond
Copy link
Contributor Author

Thank you for the reply. Commit message updated.

Please let me know if something else needs to be changed.

@JablonskiMateusz
Copy link
Contributor

one more issue, the commit type should be "test" as only test files are modified (change doesn't impact runtime behavior)

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.

2 participants