From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa1.mentor.iphmx.com (esa1.mentor.iphmx.com [68.232.129.153]) by sourceware.org (Postfix) with ESMTPS id 0CF213858D1E for ; Wed, 30 Nov 2022 18:23:02 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0CF213858D1E 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.96,207,1665475200"; d="scan'208";a="91298260" Received: from orw-gwy-01-in.mentorg.com ([192.94.38.165]) by esa1.mentor.iphmx.com with ESMTP; 30 Nov 2022 10:23:01 -0800 IronPort-SDR: eWOevjI7So7E+rS6rqbGOJJ/EmrxsnjQ8PbUtDEp42tAaecLYSCSYUDwIAnNPV8OEG5occw3sO ab1qQexNDAr+o32wYSge0bquNrl72BNOhDPpqYFniPcEDeKjGNye9Na21PVRrt0oGKigtCbPRs 6Qc7ukKzNc61UfsUu2j67q7Yf0Nbe0Gy3fv/aNocZ3qjFvL48vrThibH1Q/YTmCPAT/RUqtaTk wDpLMHCdqkJEnTcbCmFHK4COiiIUfSXWf5RqTAe5Gk0Vz0+NXqrS1HM4seLrP0thxpO8icA+lB bZ8= Date: Wed, 30 Nov 2022 18:22:56 +0000 From: Joseph Myers To: Zopolis0 CC: Subject: Re: Java front-end and library patches. In-Reply-To: Message-ID: <449467f8-19-facf-f837-c5b92c7accf0@codesourcery.com> References: <87be1195-fee5-7355-ddd-ddceedcce0a6@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: SVR-IES-MBX-08.mgc.mentorg.com (139.181.222.8) To svr-ies-mbx-10.mgc.mentorg.com (139.181.222.10) X-Spam-Status: No, score=-3109.8 required=5.0 tests=BAYES_00,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,SPF_HELO_PASS,SPF_PASS,TXREP autolearn=no 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 Wed, 30 Nov 2022, Zopolis0 via Gcc-patches wrote: > > * Each patch should have its own explanation of what it is doing and why, > > in the message body (not in an attachment). Just the commit summary line > > and ChangeLog entries aren't enough, we need the actual substantive commit > > message explaining the patch. > > The thing is, most of the patches do not need an explanation. Patches > 1-13 are just re-adding code, Then state that in the message body (with a reference to the commit that removed the code). Just because code was removed in a given form doesn't mean it should be added back in that form. For example, patch 13, "Re-add flag_evaluation_order, reorder_operands_p, and add reorder bool argument to tree_swap_operands_p", seems suspicious. That sort of global state affecting IR semantics is best avoided; rather, the Java gimplification support should deal with ensuring the correct ordering for operations in the GIMPLE generated. Note that C++ flag_strong_eval_order (for C++17 evaluation order requirements) is specific to the front end; it doesn't require anything in expr.cc or fold-const.cc or other language-independent files. So you should do something similar for Java rather than adding back global language-independent state for this. Patches 1 and 2 don't seem to have reached the mailing list. > 20-43 and 47 are just applying treewide > changes that Java missed out on, So say for each one exactly which commit it's applying the changes for. > > How has the series been validated? > > I'm not exactly sure what you mean by this. What target triplets did you run the GCC testsuite on (before and after the changes, with no regressions), with what results for the Java-specific tests? > I plan to > replace Classpath with the OpenJDK, and double down on the machine > code aspect of GCJ, dropping bytecode and interpreted support. This sort of thing is key information to include in the summary message for any future versions of the patch series. -- Joseph S. Myers joseph@codesourcery.com