[#6059] LOOP_BEGIN/LOOP_END are broken in trunk

Document created by Aaronwu Employee on Oct 17, 2013
Version 1Show Document
  • View in full screen mode

[#6059] LOOP_BEGIN/LOOP_END are broken in trunk

Submitted By: Mike Frysinger

Open Date

2010-06-04 16:36:17     Close Date

2010-06-14 11:42:18

Priority:

Medium     Assignee:

David Gibson

Board:

N/A     Silicon Revision:

Resolution:

Fixed     Fixed In Release:

N/A

Processor:

ALL     

Host Operating System:

toolchain rev.:

    kernel rev.:

State:

Closed     Found In Release:

N/A

Is this bug repeatable?:

N/A     

Summary: LOOP_BEGIN/LOOP_END are broken in trunk

Details:

 

libspeex uses LOOP_BEGIN/LOOP_END apparently and has been doing so for years.  the attached preprocessed code works fine with the 2009R1.1 toolchain but fails with trunk.  format (FLAT/FDPIC) doesnt seem to matter, so just a bug in gas.

 

2009R1.1:

$ bfin-uclinux-gcc -c test.i

$ readelf -s test.o | grep filter

    17: 00000080   480 FUNC    GLOBAL DEFAULT    1 _filter_mem16

 

trunk:

$ bfin-uclinux-gcc -c test.i

$ readelf -s test.o | grep filter

    20: 00000080   726 FUNC    GLOBAL DEFAULT    1 _filter_mem16

    21: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_start256

    22: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_start_inner256

    23: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_mid256

    24: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_mid_inner256

    28: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_start361

    29: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_start_inner361

    30: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_mid361

    31: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter_mid_inner361

    37: 00000000     0 NOTYPE  GLOBAL DEFAULT  UND filter582

 

Follow-ups

 

--- Robin Getz                                               2010-06-04 17:17:41

This is caused by:

 

------------------------------------------------------------------------

r4621 | daveg | 2010-06-03 08:48:49 -0400 (Thu, 03 Jun 2010) | 2 lines

 

   M /trunk/binutils-2.17/gas/ChangeLog.bfin

   M /trunk/binutils-2.17/gas/config/bfin-parse.y

   M /trunk/binutils-2.17/gas/config/tc-bfin.c

 

Fix for issue 5928: temporary labels for hardware loop declarations were not

supported. Also resolved a crash when using non-temporary labels.

 

so - assign to Dave.

 

Dave - when a fix is checked in -- we normally add something to the testsuite -

to make sure that we don't cause the same issue again. Can you do that for the

previous and this one?

 

-Robin

 

--- Robin Getz                                               2010-06-14 11:39:41

Fixed - and verified, so closing.

 

-Robin

 

 

 

    Files

    Changes

    Commits

    Dependencies

    Duplicates

    Associations

    Tags

 

File Name     File Type     File Size     Posted By

test.i    application/octet-stream    62911    Mike Frysinger

Attachments

Outcomes