Loading...

XML

Word

Printable

Details

Type: Task
Resolution: Done
Priority: P3: Somewhat important
Fix Version/s: None
Affects Version/s: None
Component/s: Multimedia
Labels:
None

Epic Link:
QThreadStorage
Commits:
c579277a4 (dev), 8bd163065 (6.9), 625d77949 (tqtc/lts-6.8)
Sprint:
Multimedia, weeks 15-16, Multimedia, weeks 18-19

Description

See epic for more information. Use thread_local T if the object construction is constinit (then add Q_CONSTINIT in front, too), otherwise use thread_local std::optional<T> (and also add Q_CONSTINIT).

Acceptance criteria:

All uses of QThreadStorage are removed from the module.
QT_NO_QTHREADSTORAGE is defined in .cmake.conf (cf. tasks in https://codereview.qt-project.org/q/topic:Q_FOREACH).

Attachments

Issue Links

depends on

QTBUG-117853 Add QT_NO_QTHREADSTORAGE

Reported

is duplicated by

QTBUG-133565 Investigate if thread_local should be used to store thread-local QRhi.

Closed

Gerrit Reviews

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

For Gerrit Dashboard: QTBUG-117857
#	Subject	Branch	Project	Status	CR	V
640773,7	ThreadLocalRhi: replace QThreadStorage with thread_local	dev	qt/qtmultimedia	Status: MERGED	+2	0
643120,2	ThreadLocalRhi: replace QThreadStorage with thread_local	6.9	qt/qtmultimedia	Status: MERGED	+2	0
643134,2	ThreadLocalRhi: replace QThreadStorage with thread_local	tqtc/lts-6.8	qt/tqtc-qtmultimedia	Status: MERGED	+2	0