[PATCH] D116281: [ELF] Move gotIndex/pltIndex/globalDynIndex to SymbolAux

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

There is a ~0.8% saving on max RSS.

  # current
  % repeat 5 (=time -f "\nCPU: %Us\tReal: %es\tRAM: %MKB" /tmp/c/0 -flavor gnu --threads=4 @response.txt -o /dev/shm/a.out; 'rm' -f /dev/shm/a.out) # current
  CPU: 10.26s     Real: 9.96s     RAM: 6894960KB
  CPU: 10.77s     Real: 10.01s    RAM: 6896900KB
  CPU: 10.62s     Real: 9.95s     RAM: 6892492KB
  CPU: 10.73s     Real: 9.97s     RAM: 6892396KB
  CPU: 10.86s     Real: 9.97s     RAM: 6892672KB
  
  # with D116281
  % repeat 5 (=time -f "\nCPU: %Us\tReal: %es\tRAM: %MKB" /tmp/c/1 -flavor gnu --threads=4 @response.txt -o /dev/shm/a.out; 'rm' -f /dev/shm/a.out) # current                                                                            
  CPU: 10.51s     Real: 9.72s     RAM: 6837240KB
  CPU: 10.68s     Real: 9.93s     RAM: 6839616KB
  CPU: 10.54s     Real: 9.70s     RAM: 6839244KB
  CPU: 10.57s     Real: 9.78s     RAM: 6839652KB
  CPU: 10.61s     Real: 9.91s     RAM: 6838356KB


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116281