Bug 5226 - PE-COFF targets fail to set section alignment flags correctly

Bug#:	5226 alias:	Hardware:	Reporter:	Danny Smith <dannysmith@users.sourceforge.net>
Host:		Target:	Build:
Product:			Add CC:
Component:		Version:	CC:	Remove selected CCs
Status:	RESOLVED	Priority:
Resolution:	FIXED	Severity:
Assigned To:	unassigned@sources.redhat.com	Target Milestone:

Summary:
Keywords:

Attachment	Description	Type	Created	Actions
large_align.c	Testcase for large alignment	text/plain	2007-10-29 01:07	Edit None
Create a New Attachment (proposed patch, testcase, etc.)			View All

Bug 5226 depends on:			Show dependency tree Show dependency graph
Bug 5226 blocks:			Show dependency tree Show dependency graph

Additional Comments:

Leave as RESOLVED FIXED
Reopen bug
Mark bug as VERIFIED

View Bug Activity | Format For Printing

Description:

Last confirmed: 0000-00-00 00:00

Opened: 2007-10-27 23:34

Given an alignment >16-bytes, eg

	.file	"foo.c"
	.data
.globl _x
	.align 32
_x:
	.long	1065353216
	.space 28

gas for PE-COFF targets fails to set section alignment flags correctly,
but always uses default.  So

 as foo.s | objdump -x  gives

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .text         00000000  00000000  00000000  00000000  2**4
                  CONTENTS, ALLOC, LOAD, READONLY, CODE
  1 .data         00000020  00000000  00000000  0000008c  2**4 < should be 2**5
                  ALLOC, LOAD, DATA
  2 .bss          00000000  00000000  00000000  00000000  2**4
                  ALLOC


Nor is there any warning that alignment is being set to default.

A patch for this bug has already been submitted at
http://sourceware.org/ml/binutils/2007-10/msg00325.html

------- Additional Comment #1 From Danny Smith 2007-10-29 01:07 -------

Created an attachment (id=2063)
Testcase for large alignment

------- Additional Comment #2 From Danny Smith 2007-11-04 23:01 -------

An updated patch for this bug has been submitted at
http://sourceware.org/ml/binutils/2007-11/msg00027.html

------- Additional Comment #3 From Danny Smith 2007-11-04 23:51 -------

Fixed
http://sourceware.org/ml/binutils-cvs/2007-11/msg00013.html