summaryrefslogtreecommitdiff
path: root/toolchain/binutils/patches/2.29.1/xtensa-ld-restore-old-section-sorting-behavior.patch
blob: b4118378d25d093fd4ffcef9690f3c7bc79be9f2 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
From 3a399127ead1ba3c8b8d0b9e7206fce39311ee72 Mon Sep 17 00:00:00 2001
From: Max Filippov <jcmvbkbc@gmail.com>
Date: Wed, 2 Aug 2017 02:49:56 -0700
Subject: [PATCH] xtensa: ld: restore old section sorting behavior

With the recent change 535b785fb0c9 ("Don't compare boolean values
against TRUE or FALSE") the following assertion is observed when
linking with --sort-section,alignment option:

  BFD (GNU Binutils) 2.29 assertion fail elf32-xtensa.c:3965

It appears that xtensa linker only supported sorting sections by name,
and the code that checks for the section sorting type in
ld/emultempl/xtensaelf.em was not updated in the change bcaa7b3eb957
("ld/"), that replaced boolean wildcard_spec::sorted with enumeration.

Restore the original behavior of the section sorting code.

ld/
2017-08-02  Max Filippov  <jcmvbkbc@gmail.com>

	* emultempl/xtensaelf.em
	(xtensa_wild_group_interleave_callback): Only check for by_name
	sorting.

Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
---
Backported from: 73d5923e480944e2d66bde8c59f4dff298ec57e3

 ld/emultempl/xtensaelf.em | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ld/emultempl/xtensaelf.em b/ld/emultempl/xtensaelf.em
index 1447d526a25b..3827d91b6291 100644
--- a/ld/emultempl/xtensaelf.em
+++ b/ld/emultempl/xtensaelf.em
@@ -1432,7 +1432,7 @@ xtensa_wild_group_interleave_callback (lang_statement_union_type *statement)
 	  struct wildcard_list *l;
 	  for (l = w->section_list; l != NULL; l = l->next)
 	    {
-	      if (l->spec.sorted != none)
+	      if (l->spec.sorted == by_name)
 		{
 		  no_reorder = TRUE;
 		  break;
-- 
2.1.4