diff options
author | Patrick Williams <iawillia@us.ibm.com> | 2015-08-26 07:19:18 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2015-08-26 21:43:52 -0500 |
commit | 0012c29e1b559a726b8cb2df3787d3a759974a11 (patch) | |
tree | 8e9e9ca5215dacddffbc0533bbd8e42ff012bd8e /src/build/tools | |
parent | 750a765c2f7538b7cca27749758476115868c638 (diff) | |
download | talos-hostboot-0012c29e1b559a726b8cb2df3787d3a759974a11.tar.gz talos-hostboot-0012c29e1b559a726b8cb2df3787d3a759974a11.zip |
listdeps: Ensure function order matches linker.
The listdeps.pl script did not handle correctly if two modules
exposed the same function. It was using the last module containing
the function instead of the first module. The linker script always
uses the first, so ensure the listdeps script does as well.
Change-Id: I6255033e5da9db9a4c917e70ee2e7adf0f444ad6
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20102
Tested-by: Jenkins Server
Tested-by: Jenkins OP Build CI
Tested-by: Jenkins OP HW
Reviewed-by: Zane Shelley <zshelle@us.ibm.com>
Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Diffstat (limited to 'src/build/tools')
-rwxr-xr-x | src/build/tools/listdeps.pl | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/build/tools/listdeps.pl b/src/build/tools/listdeps.pl index a4e5409d4..437202fd9 100755 --- a/src/build/tools/listdeps.pl +++ b/src/build/tools/listdeps.pl @@ -142,7 +142,10 @@ foreach $module_name (@module_names ) my $function = $values[2]; - %FunctionMap->{ $function } = $library; + if (not defined %FunctionMap->{ $function }) + { + %FunctionMap->{ $function } = $library; + } } } |