From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Native compiler - passing command line options to C compiler Date: Wed, 01 Sep 2021 19:45:06 +0300 Message-ID: <83sfyop7n1.fsf@gnu.org> References: <83bl5fkvky.fsf@gnu.org> <83v93njc3x.fsf@gnu.org> <831r69u5s3.fsf@gnu.org> <83wno0r02e.fsf@gnu.org> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="20313"; mail-complaints-to="usenet@ciao.gmane.io" Cc: emacs-devel@gnu.org, akrl@sdf.org To: Arthur Miller Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Wed Sep 01 18:55:23 2021 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mLTWD-00050q-Oj for ged-emacs-devel@m.gmane-mx.org; Wed, 01 Sep 2021 18:55:21 +0200 Original-Received: from localhost ([::1]:55342 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mLTWB-0005z0-Le for ged-emacs-devel@m.gmane-mx.org; Wed, 01 Sep 2021 12:55:19 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:41348) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mLTMd-0004jP-2p for emacs-devel@gnu.org; Wed, 01 Sep 2021 12:45:27 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:57206) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mLTMb-0007og-9B; Wed, 01 Sep 2021 12:45:25 -0400 Original-Received: from 84.94.185.95.cable.012.net.il ([84.94.185.95]:2411 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mLTMN-0006WC-It; Wed, 01 Sep 2021 12:45:25 -0400 In-Reply-To: (message from Arthur Miller on Wed, 01 Sep 2021 16:23:21 +0200) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:273671 Archived-At: > From: Arthur Miller > Cc: akrl@sdf.org, emacs-devel@gnu.org > Date: Wed, 01 Sep 2021 16:23:21 +0200 > > > Almost there: the commit log message should mention the files and > > functions where you made the changes. Look at the other log messages > > in git, and you will see the difference (feel free to ask if still > > unclear). > > > > Thanks for working on this. > > See if this is ok. I don't know how to get in ` into git message. My bash is > complaining, but I see in some git log that Lars had quoted some names with `'. > > Found also one check for WINDOWS I forgott to remove. > > See if this one is ok: > * lisp/emacs-lisp/comp.el: add user option 'native-comp-compiler-options' > and rename 'native-comp-driver-options' to 'native-comp-backend-options' > with changes needed to implement the feature and the rename. > > * list/emacs-lisp/bytecomp.el: adapt to the change in comp.el. > > * src/comp.c: add support to implement 'native-comp-compiler-options' and > the rename of driver options to backend options. This part should look like this: * lisp/emacs-lisp/comp.el ('native-comp-compiler-options): New option. (native-comp-driver-options): Rename to 'native-comp-backend-options'. * list/emacs-lisp/bytecomp.el: All users of 'native-comp-driver-options' changed. * src/comp.c (Fcomp_native_backend_options_effective_p): Renamed from 'Fcomp_native_driver_options_effective_p'; all callers changed. (Fcomp_native_compiler_options_effective_p): New function. (add_driver_options): Remove WINDOWSNT condition. (add_compiler_options): New function. (Fcomp__compile_ctxt_to_file): Call 'add_compiler_options'. IOW, you need to list not only the files, but also the functions and global variables that you change. This will be much easier if you use "C-x 4 a" or "C-c C-d" (the latter in the VC log buffer): they will generate the skeleton with file name and function name for you, so you need just add the description of the change itself, what's after the colon. (I can fix the log message for you this time, if you are exhausted ;-)