Skip to content

[backport 3.3] perf: fix a clang warning for BPS tree perftest#11147

Merged
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11123
Feb 17, 2025
Merged

[backport 3.3] perf: fix a clang warning for BPS tree perftest#11147
sergepetrenko merged 1 commit intorelease/3.3from
backport/release/3.3/11123

Conversation

@TarantoolBot
Copy link
Copy Markdown
Collaborator

@TarantoolBot TarantoolBot commented Feb 13, 2025

(This PR is a backport of #11123 to release/3.3 to a future 3.3.2 release.)


Clang warned about the getting the offset of a field in a structure
with a non-standard layout because of a std::unique_ptr field in
the DummyAllocator class. Let's simply replace it with a regular
pointer.

Closes #11122

NO_DOC=minor
NO_TEST=minor
NO_CHANGELOG=minor

Clang warned about the getting the offset of a field in a structure
with a non-standard layout because of a `std::unique_ptr` field in
the `DummyAllocator` class. Let's simply replace it with a regular
pointer.

Closes #11122

NO_DOC=minor
NO_TEST=minor
NO_CHANGELOG=minor

(cherry picked from commit 0a9258e)
@TarantoolBot TarantoolBot added the full-ci Enables all tests for a pull request label Feb 13, 2025
@TarantoolBot TarantoolBot changed the title [Backport release/3.3] perf: fix a clang warning for BPS tree perftest [backport 3.3] perf: fix a clang warning for BPS tree perftest Feb 13, 2025
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 87.391% (+0.001%) from 87.39%
when pulling ddae702 on backport/release/3.3/11123
into 0dce970
on release/3.3
.

@sergepetrenko sergepetrenko merged commit 106abf8 into release/3.3 Feb 17, 2025
60 of 74 checks passed
@sergepetrenko sergepetrenko deleted the backport/release/3.3/11123 branch February 17, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables all tests for a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants