This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/4] Add mkoffload for Intel MIC

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Thomas Schwinge <thomas at codesourcery dot com>, gcc-patches at gcc dot gnu dot org, Ilya Verbin <iverbin at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>
Cc: Kirill Yukhin <kirill dot yukhin at gmail dot com>, Andrey Turetskiy <andrey dot turetskiy at gmail dot com>
Date: Mon, 28 Sep 2015 12:09:19 +0200
Subject: Re: [PATCH 1/4] Add mkoffload for Intel MIC
Authentication-results: sourceware.org; auth=none
References: <20141021171323 dot GA47586 at msticlxl57 dot ims dot intel dot com> <20141021171602 dot GB47586 at msticlxl57 dot ims dot intel dot com> <20141022082103 dot GH10376 at tucnak dot redhat dot com> <20141022185701 dot GA21398 at msticlxl57 dot ims dot intel dot com> <87r3ljuez8 dot fsf at kepler dot schwinge dot homeip dot net> <5609106F dot 9040707 at redhat dot com>

On 09/28/2015 12:03 PM, Bernd Schmidt wrote:

On 09/28/2015 10:26 AM, Thomas Schwinge wrote:

-  objcopy_argv[8] = NULL;
+  objcopy_argv[objcopy_argc++] = NULL;
+  gcc_checking_assert (objcopy_argc <= OBJCOPY_ARGC_MAX);


On its own this is not an improvement - you're trading a compile time
error for a runtime error. So, what is the other change this is
preparing for?

Ok, I now see the other patch. But I also see that other code in thesame file and in the nvptx mkoffload is using the obstack_ptr_growmethod to build argv arrays, I think that would be preferrable to this.



Bernd

Follow-Ups:
- Re: [PATCH 1/4] Add mkoffload for Intel MIC
  - From: Ilya Verbin

References:
- Re: [PATCH 1/4] Add mkoffload for Intel MIC
  - From: Thomas Schwinge
- Re: [PATCH 1/4] Add mkoffload for Intel MIC
  - From: Bernd Schmidt

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]