* [Bug gdb/31422] Replace gdb::array_view with std::span
2024-02-27 17:05 [Bug gdb/31422] New: Replace gdb::array_view with std::span tromey at sourceware dot org
@ 2024-02-27 17:05 ` tromey at sourceware dot org
2024-02-27 17:13 ` simon.marchi at polymtl dot ca
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: tromey at sourceware dot org @ 2024-02-27 17:05 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31422
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks| |31419
--- Comment #1 from Tom Tromey <tromey at sourceware dot org> ---
Marking this as blocking c++17, even though it's somewhat
unrelated.
Referenced Bugs:
https://sourceware.org/bugzilla/show_bug.cgi?id=31419
[Bug 31419] [meta] Convert GDB to C++17
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug gdb/31422] Replace gdb::array_view with std::span
2024-02-27 17:05 [Bug gdb/31422] New: Replace gdb::array_view with std::span tromey at sourceware dot org
2024-02-27 17:05 ` [Bug gdb/31422] " tromey at sourceware dot org
@ 2024-02-27 17:13 ` simon.marchi at polymtl dot ca
2024-04-06 15:12 ` tromey at sourceware dot org
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: simon.marchi at polymtl dot ca @ 2024-02-27 17:13 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31422
Simon Marchi <simon.marchi at polymtl dot ca> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |simon.marchi at polymtl dot ca
--- Comment #2 from Simon Marchi <simon.marchi at polymtl dot ca> ---
If we want something compatible with std::span, we could use:
https://github.com/martinmoene/span-lite
I use it for another project, it works well.
Licence-wise, it's BSL (Boost), which GNU says is compatible with GPL, so it
shouldn't be a problem:
https://www.gnu.org/licenses/license-list.en.html#boost
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug gdb/31422] Replace gdb::array_view with std::span
2024-02-27 17:05 [Bug gdb/31422] New: Replace gdb::array_view with std::span tromey at sourceware dot org
2024-02-27 17:05 ` [Bug gdb/31422] " tromey at sourceware dot org
2024-02-27 17:13 ` simon.marchi at polymtl dot ca
@ 2024-04-06 15:12 ` tromey at sourceware dot org
2025-02-07 18:37 ` tromey at sourceware dot org
2025-02-07 18:39 ` tromey at sourceware dot org
4 siblings, 0 replies; 6+ messages in thread
From: tromey at sourceware dot org @ 2024-04-06 15:12 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31422
--- Comment #3 from Tom Tromey <tromey at sourceware dot org> ---
I have a patch for this, but really using std::span means
making even more changes, e.g.:
if (dwarf2_evaluate_property (&prop, nullptr, &addr_stack, &result,
{addr_stack.addr}))
return (int) (result - addr_stack.addr);
../../binutils-gdb/gdb/gnu-v3-abi.c: In function ‘int
gnuv3_baseclass_offset(type*, int, const bfd_byte*, LONGEST, CORE_ADDR, const
value*)’:
../../binutils-gdb/gdb/gnu-v3-abi.c:495:36: error: could not convert
‘{addr_stack.property_addr_info::addr}’ from ‘<brace-enclosed initializer
list>’ to ‘gdb::span<long unsigned int>’ {aka ‘std::span<long unsigned int,
18446744073709551615>’}
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug gdb/31422] Replace gdb::array_view with std::span
2024-02-27 17:05 [Bug gdb/31422] New: Replace gdb::array_view with std::span tromey at sourceware dot org
` (2 preceding siblings ...)
2024-04-06 15:12 ` tromey at sourceware dot org
@ 2025-02-07 18:37 ` tromey at sourceware dot org
2025-02-07 18:39 ` tromey at sourceware dot org
4 siblings, 0 replies; 6+ messages in thread
From: tromey at sourceware dot org @ 2025-02-07 18:37 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31422
--- Comment #4 from Tom Tromey <tromey at sourceware dot org> ---
std::span doesn't overload operator== the way that array_view does.
We'd need some kind of replacement for this.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug gdb/31422] Replace gdb::array_view with std::span
2024-02-27 17:05 [Bug gdb/31422] New: Replace gdb::array_view with std::span tromey at sourceware dot org
` (3 preceding siblings ...)
2025-02-07 18:37 ` tromey at sourceware dot org
@ 2025-02-07 18:39 ` tromey at sourceware dot org
4 siblings, 0 replies; 6+ messages in thread
From: tromey at sourceware dot org @ 2025-02-07 18:39 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=31422
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Blocks|31419 |32660
Referenced Bugs:
https://sourceware.org/bugzilla/show_bug.cgi?id=31419
[Bug 31419] [meta] Convert GDB to C++17
https://sourceware.org/bugzilla/show_bug.cgi?id=32660
[Bug 32660] Convert GDB to C++20
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread