This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
> + /* Random linker state flags. */ > +#define CR16_HASH_ENTRIES_INITIALIZED 0x1 > + char flags;
Similar questions.
The "PARAMS" and "PTR" macros are redundant. Please always use function prototypes and the "void *" type instead. ie:
#define elf32_cr16_link_hash_traverse(table, func, info) \ elf_link_hash_traverse (&(table)->root, \ (bfd_boolean (*) (struct elf_link_hash_entry *, void *)) (func), \ (info))
This applies to the other uses of PTR in the patch as well...
Similarly:
Why is this section of code suppressed ? Is it unnecessary, or something that is broken but which you intend to fix in the future ? If the code is never going to be used then it should just be removed.
You should document these new cr16 assembler operand qualifiers in gas/doc/c-cr16.texi.
with object files/libraries that use the new values. It would be better to just add the new R_CR16_GOT... relocs to the end of the current list of relocs.
Thanks Swami
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |