From patchwork Sat Aug 6 19:20:23 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Thomas Schwinge X-Patchwork-Id: 56579 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id AF8A1385C400 for ; Sat, 6 Aug 2022 19:20:54 +0000 (GMT) X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from esa1.mentor.iphmx.com (esa1.mentor.iphmx.com [68.232.129.153]) by sourceware.org (Postfix) with ESMTPS id 51FF5385697A for ; Sat, 6 Aug 2022 19:20:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 51FF5385697A Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="5.93,217,1654588800"; d="scan'208,223";a="83604386" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa1.mentor.iphmx.com with ESMTP; 06 Aug 2022 11:20:34 -0800 IronPort-SDR: Amo8/DS/m0ipyxNAxtZC52GCcKckrHBxettT2ifpC7IroSmz8MMlXmPZ/PizYN+v86ok2Vsk7G rXOuvY86807KQAcWWCTth/k/C/2Uqv7cT/MYaX3azalV9j4/2VbgVthcQjMrpWxMOYSmi67RDN oXlUN7IJLbo9r5V+Fr8AHRPeXrqfW44Ikm5YAFsGnmJdyL4WL3583EYbDWwwMpqc0XE2PQS4o+ SrWDc7Y8X1xJQaI0KypKDgJ5z91/Ife+JcJmvxqgpp3cEjOrAtDvv4GHMTEka8N/o7pMn2YpHi R+U= From: Thomas Schwinge To: Tom de Vries , Subject: [PING^6] nvptx: forward '-v' command-line option to assembler, linker In-Reply-To: <875yji36ap.fsf@dem-tschwing-1.ger.mentorg.com> References: <874k185ak4.fsf@dem-tschwing-1.ger.mentorg.com> <871qw08orn.fsf@euler.schwinge.homeip.net> <874jzv61xt.fsf@euler.schwinge.homeip.net> <878rox4d70.fsf@euler.schwinge.homeip.net> <87y1worm17.fsf@dem-tschwing-1.ger.mentorg.com> <875yji36ap.fsf@dem-tschwing-1.ger.mentorg.com> User-Agent: Notmuch/0.29.1+93~g67ed7df (https://notmuchmail.org) Emacs/26.3 (x86_64-pc-linux-gnu) Date: Sat, 6 Aug 2022 21:20:23 +0200 Message-ID: <87k07l18nc.fsf@dem-tschwing-1.ger.mentorg.com> MIME-Version: 1.0 X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: svr-ies-mbx-09.mgc.mentorg.com (139.181.222.9) To svr-ies-mbx-10.mgc.mentorg.com (139.181.222.10) X-Spam-Status: No, score=-12.0 required=5.0 tests=BAYES_00, GIT_PATCH_0, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, SPF_HELO_PASS, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org Sender: "Gcc-patches" Hi Tom! Ping. Grüße Thomas On 2022-07-27T17:48:46+0200, I wrote: > Hi Tom! > > Ping. > > > Grüße > Thomas > > > On 2022-07-20T14:44:36+0200, I wrote: >> Hi Tom! >> >> Ping. >> >> >> Grüße >> Thomas >> >> >> On 2022-07-13T10:41:23+0200, I wrote: >>> Hi Tom! >>> >>> Ping. >>> >>> >>> Grüße >>> Thomas >>> >>> >>> On 2022-07-05T16:58:54+0200, I wrote: >>>> Hi Tom! >>>> >>>> Ping. >>>> >>>> >>>> Grüße >>>> Thomas >>>> >>>> >>>> On 2022-06-07T17:41:16+0200, I wrote: >>>>> Hi! >>>>> >>>>> On 2022-05-30T09:06:21+0200, Tobias Burnus wrote: >>>>>> On 29.05.22 22:49, Thomas Schwinge wrote: >>>>>>> Not sure if that's what you had in mind, but what do you think about the >>>>>>> attached "nvptx: forward '-v' command-line option to assembler, linker"? >>>>>>> OK to push to GCC master branch (after merging >>>>>>> >>>>>>> "Put '-v' verbose output onto stderr instead of stdout")? >>>>>> >>>>>> I was mainly thinking of some way to have it available — which >>>>>> '-foffload-options=-Wa,-v' already permits on the GCC side. (Once the >>>>>> nvptx-tools patch actually makes use of the '-v'.) >>>>> >>>>> (Merged a week ago.) >>>>> >>>>>> If I understand your patch correctly, this patch now causes 'gcc -v' to >>>>>> imply 'gcc -v -Wa,-v'. I think that's okay, since 'gcc -v' already >>>>>> outputs a lot of lines and those lines can be helpful to understand what >>>>>> happens and what not. >>>>> >>>>> ACK. >>>>> >>>>>> Tom, your thoughts on this? >>>>> >>>>> Ping. >>>>> >>>>> >>>>> Grüße >>>>> Thomas ----------------- Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955 From 17c35607d4927299b0c4bd19dd6fd205c85c4a4b Mon Sep 17 00:00:00 2001 From: Thomas Schwinge Date: Sun, 29 May 2022 22:31:43 +0200 Subject: [PATCH] nvptx: forward '-v' command-line option to assembler, linker For example, for offloading compilation with '-save-temps -v', before vs. after word-diff then looks like: [...] [...]/build-gcc-offload-nvptx-none/gcc/as {+-v -v+} -o ./a.xnvptx-none.mkoffload.o ./a.xnvptx-none.mkoffload.s {+Verifying sm_30 code with sm_35 code generation.+} {+ ptxas -c -o /dev/null ./a.xnvptx-none.mkoffload.o --gpu-name sm_35 -O0+} [...] [...]/build-gcc-offload-nvptx-none/gcc/collect2 {+-v -v+} -o ./a.xnvptx-none.mkoffload [...] @./a.xnvptx-none.mkoffload.args.1 -lgomp -lgcc -lc -lgcc {+collect2 version 12.0.1 20220428 (experimental)+} {+[...]/build-gcc-offload-nvptx-none/gcc/collect-ld -v -v -o ./a.xnvptx-none.mkoffload [...] ./a.xnvptx-none.mkoffload.o -lgomp -lgcc -lc -lgcc+} {+Linking ./a.xnvptx-none.mkoffload.o as 0+} {+trying lib libc.a+} {+trying lib libgcc.a+} {+trying lib libgomp.a+} {+Resolving abort+} {+Resolving acc_on_device+} {+Linking libgomp.a::oacc-init.o/ as 1+} {+Linking libc.a::lib_a-abort.o/ as 2+} [...] (This depends on "Put '-v' verbose output onto stderr instead of stdout".) gcc/ * config/nvptx/nvptx.h (ASM_SPEC, LINK_SPEC): Define. --- gcc/config/nvptx/nvptx.h | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/gcc/config/nvptx/nvptx.h b/gcc/config/nvptx/nvptx.h index ed72c253191..b184f1d0150 100644 --- a/gcc/config/nvptx/nvptx.h +++ b/gcc/config/nvptx/nvptx.h @@ -27,6 +27,13 @@ /* Run-time Target. */ +/* Assembler supports '-v' option; handle similar to + '../../gcc.cc:asm_options', 'HAVE_GNU_AS'. */ +#define ASM_SPEC "%{v}" + +/* Linker supports '-v' option. */ +#define LINK_SPEC "%{v}" + #define STARTFILE_SPEC "%{mmainkernel:crt0.o}" #define TARGET_CPU_CPP_BUILTINS() nvptx_cpu_cpp_builtins () -- 2.25.1