From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-x42f.google.com (mail-wr1-x42f.google.com [IPv6:2a00:1450:4864:20::42f]) by sourceware.org (Postfix) with ESMTPS id E28CB3858C53 for ; Mon, 31 Jul 2023 15:55:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org E28CB3858C53 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embecosm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=embecosm.com Received: by mail-wr1-x42f.google.com with SMTP id ffacd0b85a97d-3179ed1dfbbso1394083f8f.1 for ; Mon, 31 Jul 2023 08:55:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; t=1690818913; x=1691423713; h=mime-version:user-agent:references:message-id:in-reply-to:subject :cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ybSQJMojZHmeGyyB11xgnSmhMXvu6WfhG+EXQA+wfnw=; b=ej6Nc1zoZm0ebqqaK0Y7C5uvsXfNOzN/TQCbSo7CdDc0T1Viy2uDqW0SjT+KJCVsG9 9ISxJxqwRkscA9+3MQ60SOM77S3E73UEhVdb7CHP2b6P00MspXrL0StrpIi8XZsHQsH+ ihD89icwbLpOGbD/tZqtws9gNDFjxJNdGhgmbsCvaNnE6A9qdU1F+InpNAu6LL7AzMd+ 6l7ZNNFJ+H0ob8YRto/nsZLoUfo52fXDOeyoW4CGRyQAk6IKxMiIfSNeRTTGkwpqu0bn pqLD50xlaINh5Qmdg5rPzEEZ9FrCoSLC44I3YWxcs7TPnKx16dQVgtV4VPCciO4TN6bQ /k3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690818913; x=1691423713; h=mime-version:user-agent:references:message-id:in-reply-to:subject :cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ybSQJMojZHmeGyyB11xgnSmhMXvu6WfhG+EXQA+wfnw=; b=eeIGAyKsT0pzV3tf4MEyk+YoZIeWjk9rzuZsnyLBf9hGAFYdVVD6PVfP/3YOhETBJK w3FVth2jrBzyEmhdhtpo2Ocm3DGrLw/ZmfC4nznZA6lhMhyMiOFg8givg9iaLm0vEyaf vpSHZ1qpFpw+Yzm/ojbyRnuFAWHXzzGfNThogLZdmFFbynNKLYrcFwk/4qTozZx8Eyyu egdWPEXa+ZK+tKLFbK6dx5j1RlcX7UuhF5OhxI1fjZhakyQetT1nGsBjRicHXQ0I9Ck6 jXK4CLCLJ4gXVa0EQcEG2tIpIsjFuPub5EvwSjJ6tSViSV6zGWy99e3ouzZv/s6Spf2G t7Mw== X-Gm-Message-State: ABy/qLb7pnTWGwa+TmTwiB7kKjnyTp9xc+CRubkxt5S3fnKWM9P1vh0l 65/exEUnukOHfxkWA9IarAcaEg== X-Google-Smtp-Source: APBJJlFqCub6MfXnLmA8E/YWj8k0ZjJsL7jJ42iXNKJzCJEHusF7VPDspB37EzqX+41EaR0RI2VmXA== X-Received: by 2002:a5d:6a82:0:b0:317:5ba8:ac6 with SMTP id s2-20020a5d6a82000000b003175ba80ac6mr171224wru.8.1690818912788; Mon, 31 Jul 2023 08:55:12 -0700 (PDT) Received: from tpp.orcam.me.uk (tpp.orcam.me.uk. [2001:8b0:154:0:ea6a:64ff:fe24:f2fc]) by smtp.gmail.com with ESMTPSA id l14-20020a5d668e000000b003140fff4f75sm13434677wru.17.2023.07.31.08.55.11 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 31 Jul 2023 08:55:12 -0700 (PDT) Date: Mon, 31 Jul 2023 16:55:10 +0100 (BST) From: "Maciej W. Rozycki" To: Jeff Law cc: Kito Cheng , Kito Cheng , juzhe.zhong@rivai.ai, gcc-patches , Robin Dapp , "pan2.li" Subject: Re: [PATCH] RISC-V: Return machine_mode rather than opt_machine_mode for get_mask_mode, NFC In-Reply-To: <0bede5a1-3de3-d2ac-71db-29c634e99620@gmail.com> Message-ID: References: <20230731065228.69779-1-kito.cheng@sifive.com> <0bede5a1-3de3-d2ac-71db-29c634e99620@gmail.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,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 List-Id: On Mon, 31 Jul 2023, Jeff Law wrote: > > That's a good suggestion! Thanks, let me try to apply myself workflow :) > I'm thinking that as part of the CI POC being done by RISE that the base AMI > image ought to be gcc-13 based and that we should configure the toolchains we > build with -enable-werror-always. > > While we can't necessarily get every developer to embrace this workflow, we > ought to be catching it quicker than we currently are. I wonder if we should enable the option by default, perhaps under certain conditions such as matching the build compiler version, for builds made from a Git checkout rather than a release tarball. I suspect some people are simply not aware of this option. Maciej