This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[RFA/mips] Remove mips16 code that seems redundant
- From: Joel Brobecker <brobecker at gnat dot com>
- To: Andrew Cagney <cagney at gnu dot org>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Sun, 10 Oct 2004 18:16:52 -0700
- Subject: [RFA/mips] Remove mips16 code that seems redundant
Andrew,
The block quoted in this patch was moved from mips_insn16_frame_cache()
to there (mips16_heuristic_proc_desc), and I was contemplenting the idea
of merging it the block slightly above in the same function that handles
the case when the first instruction is an entry insn. But then, looking
a bit deeper, it really seems to me that everything done in that block
is already done above. No matter how hard I look, I can't see the catch.
Is there any?
If not, I suggest removing this code. Otherwise, I'll do the merge
I thought about.
2004-10-10 Joel Brobecker <brobecker@gnat.com>
* mips-tdep.c (mips16_heuristic_proc_desc): Remove redundant code.
What do you think?
Thanks,
--
Joel