From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pj1-x102d.google.com (mail-pj1-x102d.google.com [IPv6:2607:f8b0:4864:20::102d]) by sourceware.org (Postfix) with ESMTPS id 818B03858D33 for ; Wed, 22 Feb 2023 13:00:15 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 818B03858D33 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-pj1-x102d.google.com with SMTP id d1-20020a17090a3b0100b00229ca6a4636so8919737pjc.0 for ; Wed, 22 Feb 2023 05:00:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Tm/ZWsC2XRFkWz+bHPJa+5uhC7gntr/RzwveIEIpFMM=; b=A3ey28ORbiy39KfUAxmoC9gOQ7hbFQMZy17FMjMvY6TtJW/EGEqPrj5MtulUx0KtTz XMfqa3uacNonFdxbOgg3mNQjD8ccJURTKLCoLngl2jbYdM1NtnJHwiRHx135Buv+MfT6 UTFGynpegB+/N7+R2OYQzsr7h27ye28exoVedW8dtq9l1L/gXA9GVp2L46/2XkB41Guj 6tvMFPWQqJSqhiIGUkeN3lCSIUoF3p7/z8GYNJW17tLrEEBEJ40dPXK5cG2xNb/RYXsT jhHhtimHy8Clr2EzNJ6h5lRCJkGSbGNd1Pkt47jP20ev2ZZF7Z97BbCqu0TmOdGfkhaJ +OUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Tm/ZWsC2XRFkWz+bHPJa+5uhC7gntr/RzwveIEIpFMM=; b=x9+3HyddgTHBr6Aeb/lP0lKPeL/HHav3RcThdrxGDQyvQBwjdhSRKkahDptyezVxdE 2+8LgNub4qnbanBw4dyVNNK9ETD/T80gRRDJ5lfScd2HuyJmMJpXTA/wSLxpDIDcjXib WQj2HwUs6jOtyvtdG7MIFieYBm1OM5xtXDmD8BEWDXW63HTB+JeS2nU4sXsYKi3elL8c LBVd5xr94YVDPeWneI7fe4VGrLh05tiArV83hJuHdDDE7afIAp0ctV06a4KGOJPDB+oX pEKRY+xnJwp9aWpas2HqKU3xgg51Dy4pJpa12ZV6LN7U8Z6AMQOZ67mNDC010dm+Keut lLPA== X-Gm-Message-State: AO0yUKX7JariOlqYOg+ZDqWJZYnQwl4Lk2KNRhH3OfZLkutoA8MsNQKy cX/vBjkx2kiTHCQc2MPP3s8= X-Google-Smtp-Source: AK7set9LvFzOgLdQ8G3DKSyOT5myox7ebo+d/y/fbLeTh3kR0OQS/wzvLjNfV/SzaOYqk8f8bHL4pw== X-Received: by 2002:a17:902:d4c8:b0:19b:fc1:ad99 with SMTP id o8-20020a170902d4c800b0019b0fc1ad99mr9349176plg.24.1677070812900; Wed, 22 Feb 2023 05:00:12 -0800 (PST) Received: from ?IPV6:2001:f40:91b::31ee? ([2001:f40:91b::31ee]) by smtp.gmail.com with ESMTPSA id v24-20020a17090331d800b00192c5327021sm3154206ple.200.2023.02.22.05.00.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 22 Feb 2023 05:00:12 -0800 (PST) Message-ID: <374216fc-90d1-9a49-da64-d0f1c4480cac@gmail.com> Date: Wed, 22 Feb 2023 13:00:09 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [PATCH] cygwin: Don't try to support multilibs [PR107998] To: Stromeko@NexGo.DE Cc: gcc-patches@gcc.gnu.org, Jakub Jelinek References: Content-Language: en-US From: Jonathan Yong <10walls@gmail.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3032.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 2/22/23 09:25, Jakub Jelinek wrote: > Hi! > > As discussed in the PR, t-cygwin-w64 file has been introduced in 2013 > and has one important problem, two different multilib options -m64 and -m32, > but MULTILIB_DIRNAMES with just one word in it. > Before the genmultilib sanity checking was added, my understanding is that > this essentially resulted in effective --disable-multilib, > $ gcc -print-multi-lib > .; > ;@m32 > $ gcc -print-multi-directory > . > $ gcc -print-multi-directory -m64 > . > $ gcc -print-multi-directory -m32 > > $ gcc -print-multi-os-directory > ../lib > $ gcc -print-multi-os-directory -m64 > ../lib > $ gcc -print-multi-os-directory -m32 > ../lib32 > and because of the way e.g. config-ml.in operates > multidirs= > for i in `${CC-gcc} --print-multi-lib 2>/dev/null`; do > dir=`echo $i | sed -e 's/;.*$//'` > if [ "${dir}" = "." ]; then > true > else > if [ -z "${multidirs}" ]; then > multidirs="${dir}" > else > multidirs="${multidirs} ${dir}" > fi > fi > done > dir was . first time (and so nothing was done) and empty > second time, multidirs empty too, so multidirs was set to empty > like it would be with --disable-multilib. > > With the added sanity checking the build fails unless --disable-multilib > is used in configure (dunno whether people usually configure that way > on cygwin). > >>>From what has been said in the PR, multilibs were not meant to be supported > and e.g. cygwin headers probably aren't ready for it. > > So the following patch just removes the file with the (incorrect) multilib > stuff instead of fixing it (say by setting MULTILIB_DIRNAMES to 64 32). > > I have no way to test this though, no Windows around, can anyone please > test this? I just would like to get some progress on the P1s we have... > > 2023-02-22 Jakub Jelinek > > PR target/107998 > * config.gcc (x86_64-*-cygwin*): Don't add i386/t-cygwin-w64 into > $tmake_file. > * config/i386/t-cygwin-w64: Remove. > > --- gcc/config.gcc.jj 2023-02-18 12:38:30.803025062 +0100 > +++ gcc/config.gcc 2023-02-21 17:07:12.143164563 +0100 > @@ -2105,7 +2105,7 @@ x86_64-*-cygwin*) > need_64bit_isa=yes > tm_file="${tm_file} i386/unix.h i386/bsd.h i386/gas.h i386/cygming.h i386/cygwin.h i386/cygwin-w64.h i386/cygwin-stdint.h" > xm_file=i386/xm-cygwin.h > - tmake_file="${tmake_file} i386/t-cygming t-slibgcc i386/t-cygwin-w64" > + tmake_file="${tmake_file} i386/t-cygming t-slibgcc" > target_gtfiles="$target_gtfiles \$(srcdir)/config/i386/winnt.cc" > extra_options="${extra_options} i386/cygming.opt i386/cygwin.opt" > extra_objs="${extra_objs} winnt.o winnt-stubs.o" > --- gcc/config/i386/t-cygwin-w64.jj 2020-01-12 11:54:36.333414616 +0100 > +++ gcc/config/i386/t-cygwin-w64 2023-02-21 17:06:44.121572616 +0100 > @@ -1,3 +0,0 @@ > -MULTILIB_OPTIONS = m64/m32 > -MULTILIB_DIRNAMES = 64 > -MULTILIB_OSDIRNAMES = ../lib ../lib32 > > Jakub > Achim, Mind checking this out?