[PATCH] D97204: [RFC] Clang 64-bit source locations

[Simon Tatham via Phabricator via cfe-commits]

simon_tatham added a comment.

Thanks to @miyuki for repeating the previous benchmark with this version of the patch (and on the same machine as before, which was better than I could have done).

The revised results now have the memory usage increase (compared to 32-bit SourceLocation) in the region of 5.5% to 7.5% instead of 8% to 9%. In detail:

- average memory usage during a clang build increased by 6.858 ± 0.260 %
- peak memory usage during a clang build increased by 5.529 ± 1.641 %
- memory usage during parsing of TraMP3d source file increased by 7.32 %

The change in speed between the previous version of the patch and this one is negligible (both values are within the uncertainty interval of the other one).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97204/new/

https://reviews.llvm.org/D97204