[PATCH] D32564: AArch64: compress jump tables to minimum size needed to reach destinations

[Renato Golin via Phabricator via llvm-commits]

rengolin added a comment.

In https://reviews.llvm.org/D32564#1258763, @t.p.northover wrote:

> As you'd expect, size is pretty good. Over the test-suite (including externals) nothing regresses. The total benefit over all the code is 0.5%, with notable highlights of 25% in 401.bzip2, 7% in 403.gcc, and 4% in 177.mesa.


Quite expected, but good to know the actual numbers. :)

> Time is obviously more fuzzy, but I think it's in the noise. Overall there was a 0.5% regression by geomean, but when I limited it to tests that had actually changed, that reduced to 0.05%. Other slices I took to try and improve meaning actually turned it into an improvement (including the total runtime!).

Sounds like noise to me.


Repository:
  rL LLVM

https://reviews.llvm.org/D32564