Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325678 - edc - ARM - disassembly - check for alignment when disassembling instructions
Summary: edc - ARM - disassembly - check for alignment when disassembling instructions
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-edc (show other bugs)
Version: 7.0.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Ken Ryall CLA
QA Contact: Ken Ryall CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-18 09:07 EDT by Kirk Beitz CLA
Modified: 2011-05-13 10:55 EDT (History)
0 users

See Also:

Attachments
patch for edc ARM to check for alignment (8.69 KB, patch)
2010-09-18 09:07 EDT, Kirk Beitz CLA 		cdtdoug: iplog+
Details | Diff
View All 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 Kirk Beitz CLA 2010-09-18 09:07:41 EDT 
Created attachment 179173 [details]
patch for edc ARM to check for alignment

at times, the ARM disassembler would choke on code because the code being passed was not properly aligned, given odd-address requests by DisassemblyBackendDsf retrieveDisassembly().

the attached patch was formerly part of a much larger patch in Bug 325284, but has been extracted due to that bug relying on Bug 325277.  it represents the simplest patch to HEAD as of 2010.09.17 w.r.t. performing this functionality.

ARMDisassembly.java
- modified so that getModeAndRange() attempts to align based on information it
  gets from TargetEnvironmentARM regarding ARM or Thumb mode

TargetEnironmentARM.java
- modified to make a better last-ditch guess at ARM or Thumb based upon the
  alignment of the passed address.

this patch applies cleanly to head as of 2010.09.18; a modified version of the attached patch has been tested together with other disassembly changes, and is planned for later merge together at the same time.
Comment 1 Kirk Beitz CLA 2010-10-06 22:45:45 EDT 
resolved with 2010.oct.04 edc merge commits by ken.ryall@nokia.com