S12Z [new patchset]

Message ID 20180908063004.chzt6bxgtmtycofw@jocasta.intra
State New, archived
Headers

Commit Message

John Darrington Sept. 8, 2018, 6:30 a.m. UTC
  On Sat, Sep 08, 2018 at 06:18:01AM +0200, John Darrington wrote:
          
          I still see it.  Are you sure you have an --enable-targets=all build?  What
          configure line do you use?
          
     
     I used:
     
     /home/john/binutils-gdb/configure --target=s12z --enable-targets=all CFLAGS="-g -O0" CXXFLAGS="-O0 -g" --prefix=$(mktemp -d -p /Scratch/john)
     

Now I do see it.

Can you apply the attached patch, and see if it fixes it for you?

J'
  

Comments

Simon Marchi Sept. 8, 2018, 9:37 p.m. UTC | #1
On 2018-09-08 07:30 AM, John Darrington wrote:
> On Sat, Sep 08, 2018 at 06:18:01AM +0200, John Darrington wrote:
>           
>           I still see it.  Are you sure you have an --enable-targets=all build?  What
>           configure line do you use?
>           
>      
>      I used:
>      
>      /home/john/binutils-gdb/configure --target=s12z --enable-targets=all CFLAGS="-g -O0" CXXFLAGS="-O0 -g" --prefix=$(mktemp -d -p /Scratch/john)
>      
> 
> Now I do see it.
> 
> Can you apply the attached patch, and see if it fixes it for you?
> 
> J'
> 

Yep, that seems right (and I see the patch has already been applied).

Thanks,

Simon
  

Patch

From df16439ce7de10b6f1c72c6974f544cdf806e9da Mon Sep 17 00:00:00 2001
From: John Darrington <john@darrington.wattle.id.au>
Date: Sat, 8 Sep 2018 06:59:09 +0200
Subject: [PATCH] Add ARCH_s12z to the ARCH_all case.

* opcodes/disassemble.c: Define ARCH_s12z when ARCH_all is defined.
---
 opcodes/disassemble.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/opcodes/disassemble.c b/opcodes/disassemble.c
index c5941826f0..ce83423b6d 100644
--- a/opcodes/disassemble.c
+++ b/opcodes/disassemble.c
@@ -77,6 +77,7 @@ 
 #define ARCH_rs6000
 #define ARCH_rl78
 #define ARCH_rx
+#define ARCH_s12z
 #define ARCH_s390
 #define ARCH_score
 #define ARCH_sh
-- 
2.11.0