Message ID | 20190328180543.9358-1-dimitar@dinux.eu |
---|---|
State | New, archived |
Headers |
Received: (qmail 75275 invoked by alias); 28 Mar 2019 18:06:01 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <gdb-patches.sourceware.org> List-Unsubscribe: <mailto:gdb-patches-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 75245 invoked by uid 89); 28 Mar 2019 18:06:01 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-13.7 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.1 spammy=UD:eu, H*F:D*eu, H*Ad:D*eu, HX-Get-Message-Sender-Via:sk:authent X-HELO: server28.superhosting.bg Received: from server28.superhosting.bg (HELO server28.superhosting.bg) (217.174.156.11) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 28 Mar 2019 18:05:58 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=dinux.eu; s=default; h=Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To:MIME-Version :Content-Type:Content-Transfer-Encoding:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=YQUStAUyZvkNBEcB+WpgldDykzP0YE9jY0RPYpDQIOQ=; b=NPqpCAgBexHgUdzmzBYPFW+h0I KdxrGQAt9axOoXH6y74x4St6aAxqNGHKasQuaB3XWcgk2NyzoX4m4YNVpPr0KlXDpWDSAcqYagDcq zM+KhiLo1mx8f027OKFocUNDjyBcF+rKfpmffbwAslxUd/puvJ9qYlr6IdYFSMC4qv3cSmW/7uHBU Zoz+7HyWoKUr/BzzK8feo/UmjJ8sCsOwSkoKeg9Xl8AOFELtJB5xDVOD6dxdv83sQnf/aqV4k9SKf HoM8UcoQu5ICv8xOtrcadeDE5llFPQGptK4vyUhSpoNnhV92unDLD61VZYqoNxUXtotAecebMyCZl 0SQSnpiw==; Received: from [95.87.234.74] (port=38050 helo=localhost.localdomain) by server28.superhosting.bg with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.91) (envelope-from <dimitar@dinux.eu>) id 1h9ZPW-000Ces-01; Thu, 28 Mar 2019 20:05:56 +0200 From: Dimitar Dimitrov <dimitar@dinux.eu> To: gdb-patches@sourceware.org Cc: Dimitar Dimitrov <dimitar@dinux.eu> Subject: [PATCH] sim: Fix build for avr simulator Date: Thu, 28 Mar 2019 20:05:43 +0200 Message-Id: <20190328180543.9358-1-dimitar@dinux.eu> X-OutGoing-Spam-Status: No, score=-1.0 X-IsSubscribed: yes |
Commit Message
Dimitar Dimitrov
March 28, 2019, 6:05 p.m. UTC
This commit:
commit ef9866970ce6683d40465fb7c3168f87a1dcd1b7
Date: Thu Mar 28 06:40:30 2019 +0900
sim/common: convert sim-arange to use sim-inline
Broke simulator build for AVR.
sim/avr/ChangeLog:
2019-03-28 Dimitar Dimitrov <dimitar@dinux.eu>
* interp.c: Include sim-assert.h
Signed-off-by: Dimitar Dimitrov <dimitar@dinux.eu>
---
sim/avr/interp.c | 1 +
1 file changed, 1 insertion(+)
Comments
* Dimitar Dimitrov <dimitar@dinux.eu> [2019-03-28 20:05:43 +0200]: > This commit: > > commit ef9866970ce6683d40465fb7c3168f87a1dcd1b7 > Date: Thu Mar 28 06:40:30 2019 +0900 > > sim/common: convert sim-arange to use sim-inline > > Broke simulator build for AVR. > > sim/avr/ChangeLog: > > 2019-03-28 Dimitar Dimitrov <dimitar@dinux.eu> > > * interp.c: Include sim-assert.h > > Signed-off-by: Dimitar Dimitrov <dimitar@dinux.eu> This is fine. Thanks, Andrew > --- > sim/avr/interp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sim/avr/interp.c b/sim/avr/interp.c > index 973409225a..46f0270961 100644 > --- a/sim/avr/interp.c > +++ b/sim/avr/interp.c > @@ -27,6 +27,7 @@ > #include "gdb/remote-sim.h" > > #include "sim-main.h" > +#include "sim-assert.h" > #include "sim-base.h" > #include "sim-options.h" > > -- > 2.11.0 >
On 2019-03-28 17:28, Andrew Burgess wrote: > * Dimitar Dimitrov <dimitar@dinux.eu> [2019-03-28 20:05:43 +0200]: > >> This commit: >> >> commit ef9866970ce6683d40465fb7c3168f87a1dcd1b7 >> Date: Thu Mar 28 06:40:30 2019 +0900 >> >> sim/common: convert sim-arange to use sim-inline >> >> Broke simulator build for AVR. >> >> sim/avr/ChangeLog: >> >> 2019-03-28 Dimitar Dimitrov <dimitar@dinux.eu> >> >> * interp.c: Include sim-assert.h >> >> Signed-off-by: Dimitar Dimitrov <dimitar@dinux.eu> > > This is fine. > > Thanks, > Andrew I believe that Dimitar doesn't have push access to GDB, so you should probably push the patch on his behalf. Simon
* Simon Marchi <simon.marchi@polymtl.ca> [2019-03-28 17:35:32 -0400]: > On 2019-03-28 17:28, Andrew Burgess wrote: > > * Dimitar Dimitrov <dimitar@dinux.eu> [2019-03-28 20:05:43 +0200]: > > > > > This commit: > > > > > > commit ef9866970ce6683d40465fb7c3168f87a1dcd1b7 > > > Date: Thu Mar 28 06:40:30 2019 +0900 > > > > > > sim/common: convert sim-arange to use sim-inline > > > > > > Broke simulator build for AVR. > > > > > > sim/avr/ChangeLog: > > > > > > 2019-03-28 Dimitar Dimitrov <dimitar@dinux.eu> > > > > > > * interp.c: Include sim-assert.h > > > > > > Signed-off-by: Dimitar Dimitrov <dimitar@dinux.eu> > > > > This is fine. > > > > Thanks, > > Andrew > > I believe that Dimitar doesn't have push access to GDB, so you should > probably push the patch on his behalf. Thanks for pointing this out. I've just been investigating and it turns out many other sim targets were broken by commit ef9866970ce... I think a more general fix is required, I'll post one soon. Thanks, Andrew
diff --git a/sim/avr/interp.c b/sim/avr/interp.c index 973409225a..46f0270961 100644 --- a/sim/avr/interp.c +++ b/sim/avr/interp.c @@ -27,6 +27,7 @@ #include "gdb/remote-sim.h" #include "sim-main.h" +#include "sim-assert.h" #include "sim-base.h" #include "sim-options.h"