Bug 18316 - SEC_MEP_VLIW is defined in include/elf/mep.h
Summary: SEC_MEP_VLIW is defined in include/elf/mep.h
Status: RESOLVED FIXED
Alias: None
Product: binutils
Classification: Unclassified
Component: binutils (show other bugs)
Version: 2.26
: P2 normal
Target Milestone: 2.26
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-24 12:14 UTC by H.J. Lu
Modified: 2015-04-24 12:48 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 cvs-commit@gcc.gnu.org 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.