mirror/gcc
1
0
Fork 0
mirror of https://github.com/gcc-mirror/gcc.git synced 2024-11-21 13:40:47 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

LTO plugin: modernize a bit.

Browse Source 
include/ChangeLog:

	* plugin-api.h (enum ld_plugin_tag): Do not set implicit enum
	values.

lto-plugin/ChangeLog:

	* lto-plugin.c (struct plugin_objfile): Use bool for offset
	field.
	(exec_lto_wrapper): Assign true/false to bool variables.
	(process_offload_section): Likewise.
	(claim_file_handler): Likewise.
	(onload): Likewise.
This commit is contained in:
Martin Liska 2022-01-05 10:10:19 +01:00
parent e66af28d99
commit c4ae175881
2 changed files with 44 additions and 43 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

68
include/plugin-api.h
View File

@ -487,40 +487,40 @@ enum ld_plugin_level
enum ld_plugin_tag
{
LDPT_NULL = 0,
LDPT_API_VERSION = 1,
LDPT_GOLD_VERSION = 2,
LDPT_LINKER_OUTPUT = 3,
LDPT_OPTION = 4,
LDPT_REGISTER_CLAIM_FILE_HOOK = 5,
LDPT_REGISTER_ALL_SYMBOLS_READ_HOOK = 6,
LDPT_REGISTER_CLEANUP_HOOK = 7,
LDPT_ADD_SYMBOLS = 8,
LDPT_GET_SYMBOLS = 9,
LDPT_ADD_INPUT_FILE = 10,
LDPT_MESSAGE = 11,
LDPT_GET_INPUT_FILE = 12,
LDPT_RELEASE_INPUT_FILE = 13,
LDPT_ADD_INPUT_LIBRARY = 14,
LDPT_OUTPUT_NAME = 15,
LDPT_SET_EXTRA_LIBRARY_PATH = 16,
LDPT_GNU_LD_VERSION = 17,
LDPT_GET_VIEW = 18,
LDPT_GET_INPUT_SECTION_COUNT = 19,
LDPT_GET_INPUT_SECTION_TYPE = 20,
LDPT_GET_INPUT_SECTION_NAME = 21,
LDPT_GET_INPUT_SECTION_CONTENTS = 22,
LDPT_UPDATE_SECTION_ORDER = 23,
LDPT_ALLOW_SECTION_ORDERING = 24,
LDPT_GET_SYMBOLS_V2 = 25,
LDPT_ALLOW_UNIQUE_SEGMENT_FOR_SECTIONS = 26,
LDPT_UNIQUE_SEGMENT_FOR_SECTIONS = 27,
LDPT_GET_SYMBOLS_V3 = 28,
LDPT_GET_INPUT_SECTION_ALIGNMENT = 29,
LDPT_GET_INPUT_SECTION_SIZE = 30,
LDPT_REGISTER_NEW_INPUT_HOOK = 31,
LDPT_GET_WRAP_SYMBOLS = 32,
LDPT_ADD_SYMBOLS_V2 = 33
LDPT_NULL,
LDPT_API_VERSION,
LDPT_GOLD_VERSION,
LDPT_LINKER_OUTPUT,
LDPT_OPTION,
LDPT_REGISTER_CLAIM_FILE_HOOK,
LDPT_REGISTER_ALL_SYMBOLS_READ_HOOK,
LDPT_REGISTER_CLEANUP_HOOK,
LDPT_ADD_SYMBOLS,
LDPT_GET_SYMBOLS,
LDPT_ADD_INPUT_FILE,
LDPT_MESSAGE,
LDPT_GET_INPUT_FILE,
LDPT_RELEASE_INPUT_FILE,
LDPT_ADD_INPUT_LIBRARY,
LDPT_OUTPUT_NAME,
LDPT_SET_EXTRA_LIBRARY_PATH,
LDPT_GNU_LD_VERSION,
LDPT_GET_VIEW,
LDPT_GET_INPUT_SECTION_COUNT,
LDPT_GET_INPUT_SECTION_TYPE,
LDPT_GET_INPUT_SECTION_NAME,
LDPT_GET_INPUT_SECTION_CONTENTS,
LDPT_UPDATE_SECTION_ORDER,
LDPT_ALLOW_SECTION_ORDERING,
LDPT_GET_SYMBOLS_V2,
LDPT_ALLOW_UNIQUE_SEGMENT_FOR_SECTIONS,
LDPT_UNIQUE_SEGMENT_FOR_SECTIONS,
LDPT_GET_SYMBOLS_V3,
LDPT_GET_INPUT_SECTION_ALIGNMENT,
LDPT_GET_INPUT_SECTION_SIZE,
LDPT_REGISTER_NEW_INPUT_HOOK,
LDPT_GET_WRAP_SYMBOLS,
LDPT_ADD_SYMBOLS_V2,
};
/* The plugin transfer vector. */

19
lto-plugin/lto-plugin.c
View File

@ -122,7 +122,7 @@ struct plugin_symtab
struct plugin_objfile
{
int found;
int offload;
bool offload;
simple_object_read *objfile;
struct plugin_symtab *out;
const struct ld_plugin_input_file *file;
@ -205,7 +205,7 @@ static const char *link_output_name = NULL;
/* This indicates link_output_name already contains the dot of the
suffix, so we can skip it in extensions. */
static int skip_in_suffix = 0;
static bool skip_in_suffix = false;
/* The version of gold being used, or -1 if not gold. The number is
MAJOR * 100 + MINOR. */
@ -636,7 +636,8 @@ exec_lto_wrapper (char *argv[])
/* Write argv to a file to avoid a command line that is too long
Save the file locally on save-temps. */
const char *suffix = ".lto_wrapper_args";
suffix += skip_in_suffix;
if (skip_in_suffix)
suffix++;
if (save_temps && link_output_name)
arguments_file_name = concat (link_output_name, suffix, NULL);
else
@ -1139,7 +1140,7 @@ process_offload_section (void *data, const char *name, off_t offset, off_t len)
if (startswith (name, ".gnu.offload_lto_.opts"))
{
struct plugin_objfile *obj = (struct plugin_objfile *) data;
obj->offload = 1;
obj->offload = true;
return 0;
}
@ -1182,7 +1183,7 @@ claim_file_handler (const struct ld_plugin_input_file *file, int *claimed)
*claimed = 0;
obj.file = file;
obj.found = 0;
obj.offload = 0;
obj.offload = false;
obj.out = &lto_file.symtab;
errmsg = NULL;
obj.objfile = simple_object_start_read (file->fd, file->offset, LTO_SEGMENT_NAME,
@ -1220,7 +1221,7 @@ claim_file_handler (const struct ld_plugin_input_file *file, int *claimed)
simple_object_find_sections (obj.objfile, process_offload_section,
&obj, &err);
if (obj.found == 0 && obj.offload == 0)
if (obj.found == 0 && !obj.offload)
goto err;
if (obj.found > 1)
@ -1257,10 +1258,10 @@ claim_file_handler (const struct ld_plugin_input_file *file, int *claimed)
/* If this is an LTO file without offload, and it is the first LTO file, save
the pointer to the last offload file in the list. Further offload LTO
files will be inserted after it, if any. */
if (*claimed && obj.offload == 0 && offload_files_last_lto == NULL)
if (*claimed && !obj.offload && offload_files_last_lto == NULL)
offload_files_last_lto = offload_files_last;
if (obj.offload == 1)
if (obj.offload)
{
/* Add file to the list. The order must be exactly the same as the final
order after recompilation and linking, otherwise host and target tables
@ -1547,7 +1548,7 @@ onload (struct ld_plugin_tv *tv)
assert (escapes == 0);
assert (ticks == oddticks);
assert (q - link_output_name == len - 1);
skip_in_suffix = 1;
skip_in_suffix = true;
}
}