Bug 18316

Summary: SEC_MEP_VLIW is defined in include/elf/mep.h
Product: binutils Reporter: H.J. Lu <hjl.tools>
Component: binutilsAssignee: Not yet assigned to anyone <unassigned>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 2.26   
Target Milestone: 2.26   
Host: Target:
Build: Last reconfirmed:

Description H.J. Lu 2015-04-24 12:14:00 UTC
include/elf/mep.h has

#define SEC_MEP_VLIW    SEC_TIC54X_BLOCK

which conflicts with

.#define SEC_ELF_COMPRESS 0x8000000
.
.  {* When a section with this flag is being linked, then if the size of
.     the input section is less than a page, it should not cross a page
.     boundary.  If the size of the input section is one page or more,
.     it should be aligned on a page boundary.  This is for TI
.     TMS320C54X only.  *}
.#define SEC_TIC54X_BLOCK 0x10000000
.
.  {* This section should be renamed.  This is for ELF linker
.     internal use only.  *}
.#define SEC_ELF_RENAME 0x10000000


in bfd/section.c.
Comment 1 Sourceware Commits 2015-04-24 12:42:29 UTC
The master branch has been updated by H.J. Lu <hjl@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=9361e6307bd1cf4f8f7c9de5375719ede8caa0cb

commit 9361e6307bd1cf4f8f7c9de5375719ede8caa0cb
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Fri Apr 24 05:40:23 2015 -0700

    Define SEC_MEP_VLIW in bfd/section.c
    
    Bits in section flags should only be defined in bfd/section.c.  This
    patch moves SEC_MEP_VLIW to bfd/section.c.
    
    bfd/
    
    	PR binutils/18316
    	* section.c (SEC_MEP_VLIW): New.
    	* bfd-in2.h: Regenerated.
    
    include/elf/
    
    	PR binutils/18316
    	* mep.h (SEC_MEP_VLIW): Removed.
Comment 2 H.J. Lu 2015-04-24 12:48:36 UTC
Fixed.