From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from nikam.ms.mff.cuni.cz (nikam.ms.mff.cuni.cz [195.113.20.16]) by sourceware.org (Postfix) with ESMTPS id 525ED3858401 for ; Mon, 8 Nov 2021 11:01:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 525ED3858401 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id 27C8E281D88; Mon, 8 Nov 2021 12:01:13 +0100 (CET) Date: Mon, 8 Nov 2021 12:01:13 +0100 From: Jan Hubicka To: Martin =?iso-8859-2?Q?Li=B9ka?= Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] gcov-profile: Fix -fcompare-debug with -fprofile-generate [PR100520] Message-ID: <20211108110113.GA57526@kam.mff.cuni.cz> References: <51aaa2bc-f5d7-761e-79cb-c3fb9b395f1a@suse.cz> <20211105173055.GA92445@kam.mff.cuni.cz> <20e3ce3c-c841-1263-79b4-087b40190948@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20e3ce3c-c841-1263-79b4-087b40190948@suse.cz> User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Status: No, score=-5.6 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) 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: , X-List-Received-Date: Mon, 08 Nov 2021 11:01:15 -0000 > On 11/5/21 18:30, Jan Hubicka wrote: > > every gcc source looks like bit of overkill given that is can be open > > coded in 3 statements? > > Why? It's a static inline function with few statements. I don't want to copy&paste > the same code at every location. I bet there must quite some open-coded implementations > of endswith in the GCC source code. I guess it is a matter of taste, but to me system.h should not be universal include bringing a lot of unrelated things because in long term it is how precompiled headers came to be. In theory such random generally useful things probably would belong to libiberty, but that also seems but of overkill to me. Anyway I guess we need someone with approval right to system.h to OK that. I see there is already startswith so having endwith is probably not too bad. Honza > > Martin