[PATCH] D122226: [llvm-pdbutil] Move InputFile/FormatUtil/LinePrinter to the PDB library.

[Carlos Alberto Enciso via Phabricator via llvm-commits]

CarlosAlbertoEnciso added a comment.

In D122226#3403269 <https://reviews.llvm.org/D122226#3403269>, @dblaikie wrote:

> Which build configuration is that in & could you include % growth in the chart? (it'd be interesting to see if there's any growth in a build that uses `-ffunction-sections -fdata-sections -Wl,-gc-sections`)

It is a Release configuration. I have included new data for the requested options ( `-ffunction-sections -fdata-sections -Wl,-gc-sections`)

Static libraries

| Module                | Before     | After      | % growth |
| libLLVMDebugInfoPDB.a | 2,646,354  | 2,931,710  | 10.78    |
| libLLVMSymbolize.a    | 324,626    | 324,626    | 0.00     |
| llvm-objdump          | 32,456,936 | 32,456,936 | 0.00     |
| llvm-pdbutil          | 8,084,800  | 8,090,768  | 0.07     |
| llvm-readobj          | 8,733,784  | 8,733,784  | 0.00     |
| llvm-symbolizer       | 7,201,704  | 7,201,704  | 0.00     |
| sancov                | 32,618,920 | 32,618,920 | 0.00     |
|

Shared libraries

| Module                 | Before    | After     | % growth |
| libLLVMDebugInfoPDB.so | 1,180,744 | 1,314,320 | 11.31    |
| libLLVMSymbolize.so    | 232,464   | 232,464   | 0.00     |
| llvm-objdump           | 912,696   | 912,696   | 0.00     |
| llvm-pdbutil           | 1,293,792 | 1,215,952 | -6.02    |
| llvm-readobj           | 2,168,016 | 2,168,016 | 0.00     |
| llvm-symbolizer        | 76,920    | 76,920    | 0.00     |
| sancov                 | 158,432   | 158,432   | 0.00     |


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

https://reviews.llvm.org/D122226