diff options
| author | krause <krause@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2011-01-17 07:50:42 +0000 |
|---|---|---|
| committer | krause <krause@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2011-01-17 07:50:42 +0000 |
| commit | 363dee978e43fd6ca776c2d43fe6b12fec4e3911 (patch) | |
| tree | 9de9787987eb6a3374e18835b196b7ac6cb9ad19 /cli_classic.c | |
| parent | 8afaf634a5b34fcd9794ac201819af4010611ef1 (diff) | |
| download | flashrom-363dee978e43fd6ca776c2d43fe6b12fec4e3911.tar.gz | |
This patch reduces the stack usage by declaring 'const' stack variables
as 'static const' so they end up in the .rodata section instead of being
copied from there to the stack for every invocation of the corresponding
function. As a plus we end up in having a smaller binary as the "copy
from .rodata to stack" code isn't emitted by the compiler any more
(roughly -100 bytes).
Signed-off-by: Mathias Krause <mathias.krause@secunet.com>
Acked-by: Stefan Reinauer <stepan@coreboot.org>
git-svn-id: https://code.coreboot.org/svn/flashrom/trunk@1252 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'cli_classic.c')
| -rw-r--r-- | cli_classic.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cli_classic.c b/cli_classic.c index 9b9b451..1211989 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -114,8 +114,8 @@ int cli_classic(int argc, char *argv[]) int operation_specified = 0; int i; - const char *optstring = "r:Rw:v:nVEfc:m:l:i:p:Lzh"; - static struct option long_options[] = { + static const char optstring[] = "r:Rw:v:nVEfc:m:l:i:p:Lzh"; + static const struct option long_options[] = { {"read", 1, 0, 'r'}, {"write", 1, 0, 'w'}, {"erase", 0, 0, 'E'}, |