Hello-

Here is an upated patch series. Most of the patches have not changed from v1.

v1 thread: https://gcc.gnu.org/pipermail/gcc-patches/2024-November/667333.html

The changes are:

- As Richard suggested, I changed to make the feature not configurable,
  since it will be too hard to optimize structure packing without committing
  to the size of location_t. In v1, the first patch of the series added the
  support and the last patch in the series added the configure option. Now,
  the first patch lays the groundwork in libcpp to make the size of
  location_t easy to adjust, and the last patch in the series actually
  changes it. This way, the series could still be committed as all separate
  patches without creating any invalid intermediate states.

- Two of the v1 patches were acked by Jason. One of those (v1 05/15) has
  already been pushed as r15-5067 and does not appear in v2. The other one
  (v1 10/15) has changed very slightly in v2 and is now v2 09/14. The only
  change is the #ifdef on the former configure option has been replaced with
  a check on sizeof(location_t) instead, since the configure option is no
  longer in use.

- There was some discussion on the idea of the patches that I took to
  indicate general support for moving in this direction. I did not yet
  receive any specific feedback on the rest of the patches.

Thanks!

-Lewis

01/14 Support for 64-bit location_t: libcpp part 1
02/14 libcpp: Fix potential unaligned access in cpp_buffer
03/14 tree-cfg: Fix call to next_discriminator_for_locus()
04/14 tree-phinodes: Use 4 instead of 2 as the minimum number of phi args
05/14 gimple: Handle tail padding when computing gimple_ops_offset
06/14 Support for 64-bit location_t: Frontend parts
07/14 Support for 64-bit location_t: toplev parts
08/14 Support for 64-bit location_t: Analyzer parts
09/14 Support for 64-bit location_t: C++ modules parts
10/14 Support for 64-bit location_t: gimple parts
11/14 Support for 64-bit location_t: RTL parts
12/14 Support for 64-bit location_t: Backend parts
13/14 Support for 64-bit location_t: Internal parts
14/14 Support for 64-bit location_t: Final parts

 59 files changed, 709 insertions(+), 427 deletions(-)

Reply via email to