Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/make make(1): reduce code size of parse_debug_options
details: https://anonhg.NetBSD.org/src/rev/6069a4cf06f6
branches: trunk
changeset: 946067:6069a4cf06f6
user: rillig <rillig%NetBSD.org@localhost>
date: Sun Nov 15 08:10:15 2020 +0000
description:
make(1): reduce code size of parse_debug_options
Each access to a global variable requires a relocation. Skip these.
diffstat:
usr.bin/make/main.c | 50 ++++++++++++++++++++++++++------------------------
1 files changed, 26 insertions(+), 24 deletions(-)
diffs (133 lines):
diff -r 58bce548cec3 -r 6069a4cf06f6 usr.bin/make/main.c
--- a/usr.bin/make/main.c Sun Nov 15 08:09:56 2020 +0000
+++ b/usr.bin/make/main.c Sun Nov 15 08:10:15 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: main.c,v 1.469 2020/11/14 23:03:08 rillig Exp $ */
+/* $NetBSD: main.c,v 1.470 2020/11/15 08:10:15 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -109,7 +109,7 @@
#include "trace.h"
/* "@(#)main.c 8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: main.c,v 1.469 2020/11/14 23:03:08 rillig Exp $");
+MAKE_RCSID("$NetBSD: main.c,v 1.470 2020/11/15 08:10:15 rillig Exp $");
#if defined(MAKE_NATIVE) && !defined(lint)
__COPYRIGHT("@(#) Copyright (c) 1988, 1989, 1990, 1993 "
"The Regents of the University of California. "
@@ -245,83 +245,84 @@
parse_debug_options(const char *argvalue)
{
const char *modules;
+ DebugFlags debug = opts.debug;
for (modules = argvalue; *modules; ++modules) {
switch (*modules) {
case '0': /* undocumented, only intended for tests */
- opts.debug = DEBUG_NONE;
+ debug = DEBUG_NONE;
break;
case 'A':
- opts.debug = DEBUG_ALL;
+ debug = DEBUG_ALL;
break;
case 'a':
- opts.debug |= DEBUG_ARCH;
+ debug |= DEBUG_ARCH;
break;
case 'C':
- opts.debug |= DEBUG_CWD;
+ debug |= DEBUG_CWD;
break;
case 'c':
- opts.debug |= DEBUG_COND;
+ debug |= DEBUG_COND;
break;
case 'd':
- opts.debug |= DEBUG_DIR;
+ debug |= DEBUG_DIR;
break;
case 'e':
- opts.debug |= DEBUG_ERROR;
+ debug |= DEBUG_ERROR;
break;
case 'f':
- opts.debug |= DEBUG_FOR;
+ debug |= DEBUG_FOR;
break;
case 'g':
if (modules[1] == '1') {
- opts.debug |= DEBUG_GRAPH1;
+ debug |= DEBUG_GRAPH1;
++modules;
} else if (modules[1] == '2') {
- opts.debug |= DEBUG_GRAPH2;
+ debug |= DEBUG_GRAPH2;
++modules;
} else if (modules[1] == '3') {
- opts.debug |= DEBUG_GRAPH3;
+ debug |= DEBUG_GRAPH3;
++modules;
}
break;
case 'h':
- opts.debug |= DEBUG_HASH;
+ debug |= DEBUG_HASH;
break;
case 'j':
- opts.debug |= DEBUG_JOB;
+ debug |= DEBUG_JOB;
break;
case 'L':
opts.lint = TRUE;
break;
case 'l':
- opts.debug |= DEBUG_LOUD;
+ debug |= DEBUG_LOUD;
break;
case 'M':
- opts.debug |= DEBUG_META;
+ debug |= DEBUG_META;
break;
case 'm':
- opts.debug |= DEBUG_MAKE;
+ debug |= DEBUG_MAKE;
break;
case 'n':
- opts.debug |= DEBUG_SCRIPT;
+ debug |= DEBUG_SCRIPT;
break;
case 'p':
- opts.debug |= DEBUG_PARSE;
+ debug |= DEBUG_PARSE;
break;
case 's':
- opts.debug |= DEBUG_SUFF;
+ debug |= DEBUG_SUFF;
break;
case 't':
- opts.debug |= DEBUG_TARG;
+ debug |= DEBUG_TARG;
break;
case 'V':
opts.debugVflag = TRUE;
break;
case 'v':
- opts.debug |= DEBUG_VAR;
+ debug |= DEBUG_VAR;
break;
case 'x':
- opts.debug |= DEBUG_SHELL;
+ debug |= DEBUG_SHELL;
break;
case 'F':
parse_debug_option_F(modules + 1);
@@ -333,6 +334,7 @@
usage();
}
}
+ opts.debug = debug;
debug_setbuf:
/*
Home |
Main Index |
Thread Index |
Old Index