[PATCH 4/4] ld: Add ldlang_check_relro_region/update lang_find_relro_sections_1

Alan Modra amodra@gmail.com
Mon Nov 13 00:12:00 GMT 2017


On Sun, Nov 12, 2017 at 09:31:51AM -0800, H.J. Lu wrote:
> 	(ldlang_check_relro_region): New function.
> 	(lang_find_relro_sections_1): Add an argument for pointer to

This is a nitpick, but most other functions that have a
lang_statement_union_type* argument have it as the first parameter.
It obviously doesn't matter, but I think it would be nicer for
consistency if you kept the same parameter order here.

Other than that, the patch series looks fine to me.

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list