[v2,1/2] ISA bit treatment on the MIPS platform
Commit Message
Joel, Doug --
On Thu, 4 Dec 2014, Maciej W. Rozycki wrote:
> > This is not a requirement for your patch, but if you happen to be able
> > to quickly document elf_make_msymbol_special as well, that would be
> > a very welcome and appreciated change.
>
> I have now added these descriptions, with some bias towards the MIPS
> specifics as they are what I am most familiar with. Please let me know
> if you think they might be further improved.
I haven't heard back from you, so I have applied the change now.
While reviewing the final ChangeLog entry I discovered I created an
unneeded shadow local variable definition, so in addition to the changes
already posted in this thread I have included the following cleanup, as
obvious, in the final commit.
Thanks for your review.
Maciej
gdb-mips16-isa-bit-cleanup.diff
Comments
On Fri, Dec 12, 2014 at 6:00 AM, Maciej W. Rozycki
<macro@codesourcery.com> wrote:
> Joel, Doug --
>
> On Thu, 4 Dec 2014, Maciej W. Rozycki wrote:
>
>> > This is not a requirement for your patch, but if you happen to be able
>> > to quickly document elf_make_msymbol_special as well, that would be
>> > a very welcome and appreciated change.
>>
>> I have now added these descriptions, with some bias towards the MIPS
>> specifics as they are what I am most familiar with. Please let me know
>> if you think they might be further improved.
>
> I haven't heard back from you, so I have applied the change now.
>
> While reviewing the final ChangeLog entry I discovered I created an
> unneeded shadow local variable definition, so in addition to the changes
> already posted in this thread I have included the following cleanup, as
> obvious, in the final commit.
Hi.
Thanks very much for the added comments.
===================================================================
@@ -7027,9 +7027,6 @@ add_partial_subprogram (struct partial_d
CORE_ADDR *lowpc, CORE_ADDR *highpc,
int set_addrmap, struct dwarf2_cu *cu)
{
- struct objfile *objfile = cu->objfile;
- struct gdbarch *gdbarch = get_objfile_arch (objfile);
-
if (pdi->tag == DW_TAG_subprogram)
{
if (pdi->has_pc_info)
@@ -7041,6 +7038,7 @@ add_partial_subprogram (struct partial_d
if (set_addrmap)
{
struct objfile *objfile = cu->objfile;
+ struct gdbarch *gdbarch = get_objfile_arch (objfile);
CORE_ADDR baseaddr;
CORE_ADDR highpc;
CORE_ADDR lowpc;