about summary refs log tree commit diff stats
path: root/arc/.traces/get-address
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2016-08-31 14:56:33 -0700
committerKartik K. Agaram <vc@akkartik.com>2016-08-31 15:07:03 -0700
commit274ed0fc85e7c50fc0e551c5e376aebdb9a548f4 (patch)
tree96c38529edf951e963a27d19c55caa47133e6c8a /arc/.traces/get-address
parent0708d07f905687e38d4b2544117eeccbcda9d539 (diff)
downloadmu-274ed0fc85e7c50fc0e551c5e376aebdb9a548f4.tar.gz
3288 - cleaner heuristic for cleaving
For the last couple of days I've been implicitly thinking in terms of
how many compilation units I want to generate. Might as well make that
explicit and drop the hacky ideas for approximating it.

I tried more timing experiments like the ones in commit 3281.
Conclusion: I can't have the best of both worlds:

  1. Full compilation doesn't take too much longer than with a single
  compilation unit.

  2. Incremental compilation is fast enough that there's negligible
  benefit from dropping optimization.

We're still taking on a 10s hit in full build time.

I care more about not degrading the full compilation too much, since
that gets magnified so much on the Couch's puny server. So we'll just
have to continue using CXXFLAGS=-g when we care to save a few seconds in
incremental compilation time.

A final mystery: the build time increases by 10s with the new heuristic
even though the number of calls to the compiler (and therefore the fixed
cost) is the same. Seems like separating certain functions into
different units is causing the compiler issues. Dropping from 4 to 3
compilation units eliminated the issue.

--- Appendix: Measurements
before:
  full build 4 + test: 42s
  incremental compilation with -O3: varied from 30s for mu_0.cc to 5s for mu_3.cc
    longer times benefitted from dropping -O3

after:
  full build 1 + test: 39s
  full build 2 + test: 41s
  full build 3 + test: 43s
  full build 4 + test: 52s
  full build 5 + test: 53s
  full build 6 + test: 51s
  full build 10 (9) + test: 54s
  full build 20 (16) + test: 58s
Diffstat (limited to 'arc/.traces/get-address')
0 files changed, 0 insertions, 0 deletions
> 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219