Configure gdb for nacl target

Configure gdb for nacl target

BUG= none
TEST= none

Committed: https://git.chromium.org/gitweb?p=native_client/nacl-gcc.git;a=commit;h=4b7b669

[eaeltsin, 2012-05-05 16:16:50]

Can you please add more details in the changeset description?

Probably we only need x86_64-nacl-gdb, it is capable of debugging i686 code as
well, and as we are supporting remote debugging only, we can compile
x86_64-nacl-gdb as 32-bit binary.

[eaeltsin, 2012-05-05 16:20:42]

https://chromiumcodereview.appspot.com/10365030/diff/5001/bfd/config.bfd
File bfd/config.bfd (right):

https://chromiumcodereview.appspot.com/10365030/diff/5001/bfd/config.bfd#newc...
bfd/config.bfd:576: targ64_selvecs="bfd_elf64_x86_64_vec bfd_elf32_x86_64_vec
bfd_elf64_l1om_vec bfd_elf64_k1om_vec"
We probably don't need this target.

https://chromiumcodereview.appspot.com/10365030/diff/5001/gdb/configure.tgt
File gdb/configure.tgt (right):

https://chromiumcodereview.appspot.com/10365030/diff/5001/gdb/configure.tgt#n...
gdb/configure.tgt:227: *nacl*)
We need to be more specific here - imagine we want to add nacl gdb for ARM and
other architectures, this case should not fire.

[halyavin, 2012-05-10 08:21:41]

https://chromiumcodereview.appspot.com/10365030/diff/5001/gdb/configure.tgt
File gdb/configure.tgt (right):

https://chromiumcodereview.appspot.com/10365030/diff/5001/gdb/configure.tgt#n...
gdb/configure.tgt:227: *nacl*)
On 2012/05/05 16:20:47, eaeltsin wrote:
> We need to be more specific here - imagine we want to add nacl gdb for ARM and
> other architectures, this case should not fire.

Done.

[halyavin, 2012-05-10 09:23:31]

https://chromiumcodereview.appspot.com/10365030/diff/5001/bfd/config.bfd
File bfd/config.bfd (right):

https://chromiumcodereview.appspot.com/10365030/diff/5001/bfd/config.bfd#newc...
bfd/config.bfd:576: targ64_selvecs="bfd_elf64_x86_64_vec bfd_elf32_x86_64_vec
bfd_elf64_l1om_vec bfd_elf64_k1om_vec"
On 2012/05/05 16:20:47, eaeltsin wrote:
> We probably don't need this target.

Done.

[eaeltsin, 2012-05-10 09:28:30]

lgtm with one change below.

https://chromiumcodereview.appspot.com/10365030/diff/10001/gdb/configure.tgt
File gdb/configure.tgt (right):

https://chromiumcodereview.appspot.com/10365030/diff/10001/gdb/configure.tgt#...
gdb/configure.tgt:227: i[34567]86-*-nacl* | x86_64-*-nacl*)
As we discussed in person, you don't need i[34567]86-*-nacl* at all, we should
not configure for this target.

Please keep only x86_64-*-nacl* here.

[halyavin, 2012-05-10 10:09:00]

https://chromiumcodereview.appspot.com/10365030/diff/10001/gdb/configure.tgt
File gdb/configure.tgt (right):

https://chromiumcodereview.appspot.com/10365030/diff/10001/gdb/configure.tgt#...
gdb/configure.tgt:227: i[34567]86-*-nacl* | x86_64-*-nacl*)
On 2012/05/10 09:28:30, eaeltsin wrote:
> As we discussed in person, you don't need i[34567]86-*-nacl* at all, we should
> not configure for this target.
> 
> Please keep only x86_64-*-nacl* here.

Done.