Re: [PATCH] D24933: Enable configuration files in clang

2017-08-07 Thread Serge Pavlov via cfe-commits
2017-08-07 1:46 GMT+07:00 Richard Smith : > On 6 August 2017 at 11:15, Serge Pavlov via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> 2017-08-06 6:43 GMT+07:00 Hal Finkel : >> >>> On 07/24/2017 10:18 AM, Serge Pavlov wrote: >>> >>> I am thinking about reducing the patch further to leave

Re: [PATCH] D24933: Enable configuration files in clang

2017-08-06 Thread Richard Smith via cfe-commits
On 6 August 2017 at 11:15, Serge Pavlov via cfe-commits < cfe-commits@lists.llvm.org> wrote: > 2017-08-06 6:43 GMT+07:00 Hal Finkel : > >> On 07/24/2017 10:18 AM, Serge Pavlov wrote: >> >> I am thinking about reducing the patch further to leave only the ability >> to include config file when clang

Re: [PATCH] D24933: Enable configuration files in clang

2017-08-06 Thread Hal Finkel via cfe-commits
On 08/06/2017 01:15 PM, Serge Pavlov wrote: 2017-08-06 6:43 GMT+07:00 Hal Finkel >: On 07/24/2017 10:18 AM, Serge Pavlov wrote: I am thinking about reducing the patch further to leave only the ability to include config file when clang is called as `targ

Re: [PATCH] D24933: Enable configuration files in clang

2017-08-06 Thread Serge Pavlov via cfe-commits
2017-08-06 6:43 GMT+07:00 Hal Finkel : > On 07/24/2017 10:18 AM, Serge Pavlov wrote: > > I am thinking about reducing the patch further to leave only the ability > to include config file when clang is called as `target-clang-drivermode`. > It is still useful for cross compilation tasks because: >

Re: [PATCH] D24933: Enable configuration files in clang

2017-08-05 Thread Hal Finkel via cfe-commits
On 07/24/2017 10:18 AM, Serge Pavlov wrote: I am thinking about reducing the patch further to leave only the ability to include config file when clang is called as `target-clang-drivermode`. It is still useful for cross compilation tasks because: - It is a convenient way to switch between sup

Re: [PATCH] D24933: Enable configuration files in clang

2017-07-24 Thread Serge Pavlov via cfe-commits
I am thinking about reducing the patch further to leave only the ability to include config file when clang is called as `target-clang-drivermode`. It is still useful for cross compilation tasks because: - It is a convenient way to switch between supported targets, - SDK producer can ship compiler w

Re: [PATCH] D24933: Enable configuration files in clang

2017-05-09 Thread Serge Pavlov via cfe-commits
2017-05-10 3:46 GMT+07:00 Richard Smith : > On 1 March 2017 at 02:50, Serge Pavlov via Phabricator < > revi...@reviews.llvm.org> wrote: > >> >> Format of configuration file is similar to file used in the construct >> `@file`, it is a set of options. Configuration file have advantage over >> this c

Re: [PATCH] D24933: Enable configuration files in clang

2017-05-09 Thread Richard Smith via cfe-commits
On 1 March 2017 at 02:50, Serge Pavlov via Phabricator < revi...@reviews.llvm.org> wrote: > sepavloff added a comment. > > Glad to know that someone is interested in this feature! > Below is actual proposal. > > **Adding named configuration files to clang driver** > > A configuration file is a col

Re: [PATCH] D24933: Enable configuration files in clang

2016-09-26 Thread Daniel Dunbar via cfe-commits
ddunbar added a comment. I am too out of the loop on Clang development to be able to comment on the specific direction, but I will just say that I am highly in favor of adding new features in this direction. Thank you! https://reviews.llvm.org/D24933