Details
-
Suggestion
-
Resolution: Invalid
-
P2: Important
-
5.3.2, 5.4.2, 5.5.1, 5.6.2, 5.7.1, 5.8.0, 5.9
-
None
Description
Please change the length parameter of the qt_ucstrncmp_mips_dsp_asm() function from uint (always 32 bits) to size_t (32 or 64 bits, depending on the processor's word size).
The introduction of QStringView (https://codereview.qt-project.org/143763) necessitates extending internal string handling functions to be able to take more than 2 Gi of characters, so I'll likely need the same change in the from/toLatin1 functions.
I will work around the issue by looping over the functions in blocks of UINT_MAX, but it would be nice if you could change the assembler functions eventually.
Attachments
For Gerrit Dashboard: QTBUG-59150 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
422998,4 | Statically assert that the __mips_dsp code is only active on 32-bit | dev | qt/qtbase | Status: MERGED | +2 | 0 |
424976,2 | Statically assert that the __mips_dsp code is only active on 32-bit | 6.4 | qt/qtbase | Status: MERGED | +2 | 0 |
424977,2 | Statically assert that the __mips_dsp code is only active on 32-bit | 6.3 | qt/qtbase | Status: MERGED | +2 | 0 |
424991,2 | Statically assert that the __mips_dsp code is only active on 32-bit | tqtc/lts-6.2 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |
424992,2 | Statically assert that the __mips_dsp code is only active on 32-bit | tqtc/lts-5.15 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |