Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328870 - LLong type has wrong alignment for Arm targets
Summary: LLong type has wrong alignment for Arm targets
Status: CLOSED FIXED
Alias: None
Product: RTSC
Classification: Technology
Component: Targets (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Sasha Slijepcevic CLA
QA Contact:
URL:
Whiteboard: target:3.20.05
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-27 14:06 EDT by Chris McCormick CLA
Modified: 2011-06-20 14:39 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris McCormick CLA 2010-10-27 14:06:26 EDT 
ti/targets/arm/IArm.xdc currently has the alignment for the ‘t_LLong' type as ‘4’, but it needs to be ‘8’.

The ROV test DecoderTest1 uncovered this.
Comment 1 Sasha Slijepcevic CLA 2010-10-29 15:05:57 EDT 
- fixed in xdctargets-c31

I ran basic.size example for all Arm COFF and ELF TI targets to confirm that 8 byte types have the alignment of 4 on COFF, and 8 on ELF.
Comment 2 Alan DeMars CLA 2010-12-02 13:17:12 EST 
I reviewed the modifications to arm.elf.IArm.xdc and am satisfied with the changes.
Comment 3 Dave Russo CLA 2010-12-06 11:51:44 EST 
Verified by Alan DeMars


From: DeMars, Alan 
Sent: Thursday, December 02, 2010 10:18 AM
To: Mookerjee, Amit; Wechsler, Karl
Cc: Mullanix, Todd; Prasad, Shreyas; Slijepcevic, Sasa
Subject: RE: Assertion with BIOS version 6.31..00.18

Done.

________________________________________
From: Mookerjee, Amit 
Sent: Thursday, December 02, 2010 9:23 AM
To: Wechsler, Karl; DeMars, Alan
Cc: Mullanix, Todd; Prasad, Shreyas; Slijepcevic, Sasa
Subject: RE: Assertion with BIOS version 6.31..00.18
Karl,
The “Whiteboard:” field for the bug is not populated. So it was not picked up in the release notes and also missed out on the fix verify list for the release. 
Alan – If you are satisfied with the fix can you put a note in the bug report? I will change the bug state to VERIFIED.

>The relnotes archive directory for 3.20.05 is missing the relnotes for 3.20.02, 3.20.03, and 3.20.04.  Can you please fix this for 3.20.06? 

OK


Regards
Amit
________________________________________
From: Wechsler, Karl 
Sent: Thursday, December 02, 2010 8:58 AM
To: DeMars, Alan
Cc: Mullanix, Todd; Prasad, Shreyas; Slijepcevic, Sasa; Mookerjee, Amit
Subject: RE: Assertion with BIOS version 6.31..00.18

Thanks.

Amit – 

(1)  Any ideas how this didn’t get picked up in the 3.20.05 release notes?   This change was made between 3.20.04 and 3.20.05 GA releases.

(2)  The relnotes archive directory for 3.20.05 is missing the relnotes for 3.20.02, 3.20.03, and 3.20.04.  Can you please fix this for 3.20.06?  Thanks.

-Karl-

________________________________________
From: DeMars, Alan 
Sent: Thursday, December 02, 2010 8:44 AM
To: Wechsler, Karl
Cc: Mullanix, Todd; Prasad, Shreyas; Slijepcevic, Sasa
Subject: RE: Assertion with BIOS version 6.31..00.18

ECL328870
Comment 4 Dave Russo CLA 2011-06-20 14:39:09 EDT 
Shipped in XDCtools 3.20.05