diff options
| author | Paul Eggert | 2019-04-18 00:30:24 -0700 |
|---|---|---|
| committer | Paul Eggert | 2019-04-18 00:35:18 -0700 |
| commit | 6d6c55db2cdfb6b354873f17285a3f602e011817 (patch) | |
| tree | fc9647797be220e613962260700c05e5f37100c1 /src/json.c | |
| parent | dded2c4cf30fbdb5e90c44bc76a26970d00e0f22 (diff) | |
| download | emacs-6d6c55db2cdfb6b354873f17285a3f602e011817.tar.gz emacs-6d6c55db2cdfb6b354873f17285a3f602e011817.zip | |
Mark _Noreturn error functions as cold
On my platform this made ‘make compile-always’ 1.3% faster.
Suggested by Alex Gramiak in:
https://lists.gnu.org/r/emacs-devel/2019-04/msg00684.html
* configure.ac (nw): Don’t use -Wsuggest-attribute=cold.
* lib-src/make-docfile.c (write_globals):
Mark noreturn functions as cold.
* src/callproc.c (exec_failed):
* src/data.c (wrong_length_argument, wrong_type_argument):
* src/emacs-module.c (module_abort):
* src/emacs.c (terminate_due_to_signal):
* src/eval.c (unwind_to_catch):
* src/image.c (my_png_error, my_error_exit):
* src/json.c (json_out_of_memory, json_parse_error):
* src/keyboard.c (quit_throw_to_read_char, user_error):
* src/lisp.h (die, wrong_type_argument, wrong_choice)
(args_out_of_range, args_out_of_range_3, circular_list)
(buffer_overflow, memory_full, buffer_memory_full)
(string_overflow, xsignal, xsignal0, xsignal1, xsignal2)
(xsignal3, signal_error, overflow_error, error, verror)
(nsberror, report_file_errno, report_file_error)
(report_file_notify_error, terminate_due_to_signal)
(emacs_abort, fatal):
* src/lread.c (load_error_old_style_backquotes)
(end_of_file_error, invalid_syntax):
* src/pdumper.c (error_unsupported_dump_object):
* src/puresize.h (pure_write_error):
* src/search.c (matcher_overflow):
* src/sound.c (sound_perror, alsa_sound_perror):
* src/sysdep.c (handle_arith_signal):
* src/systime.h (time_overflow):
* src/term.c (maybe_fatal, vfatal):
* src/textprop.c (text_read_only):
* src/timefns.c (invalid_time_zone_specification)
(time_error, invalid_hz):
* src/xterm.c (x_connection_closed):
Use AVOID instead of _Noreturn void, so that it’s marked cold.
* src/conf_post.h (__has_attribute_cold) [!__has_attribute]:
New macro.
(ATTRIBUTE_COLD): New macro.
* src/frame.h (WINDOW_SYSTEM_RETURN): Add ATTRIBUTE_COLD.
* src/lisp.h (AVOID): New macro.
* src/xterm.c: Omit unnecessary static decls, so that we needn’t
worry about which functions should be marked cold.
(x_io_error_quitter): Mark as cold.
Diffstat (limited to 'src/json.c')
| -rw-r--r-- | src/json.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/json.c b/src/json.c index 74e0534065f..5917212899e 100644 --- a/src/json.c +++ b/src/json.c | |||
| @@ -255,7 +255,7 @@ json_encode (Lisp_Object string) | |||
| 255 | return code_convert_string (string, Qutf_8_unix, Qt, true, true, true); | 255 | return code_convert_string (string, Qutf_8_unix, Qt, true, true, true); |
| 256 | } | 256 | } |
| 257 | 257 | ||
| 258 | static _Noreturn void | 258 | static AVOID |
| 259 | json_out_of_memory (void) | 259 | json_out_of_memory (void) |
| 260 | { | 260 | { |
| 261 | xsignal0 (Qjson_out_of_memory); | 261 | xsignal0 (Qjson_out_of_memory); |
| @@ -263,7 +263,7 @@ json_out_of_memory (void) | |||
| 263 | 263 | ||
| 264 | /* Signal a Lisp error corresponding to the JSON ERROR. */ | 264 | /* Signal a Lisp error corresponding to the JSON ERROR. */ |
| 265 | 265 | ||
| 266 | static _Noreturn void | 266 | static AVOID |
| 267 | json_parse_error (const json_error_t *error) | 267 | json_parse_error (const json_error_t *error) |
| 268 | { | 268 | { |
| 269 | Lisp_Object symbol; | 269 | Lisp_Object symbol; |