Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-16 Thread Manuel Klimek via cfe-commits
Thanks! On Wed, Sep 16, 2015 at 11:30 AM Argyrios Kyrtzidis wrote: > In r247832. > > On Sep 12, 2015, at 5:19 AM, Manuel Klimek wrote: > > Test would be awesome :) Thx! > > On Fri, Sep 11, 2015 at 10:44 PM Argyrios Kyrtzidis > wrote: > >> In r247468, thanks for reviewing! >> >> On Sep 11, 2015

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-16 Thread Argyrios Kyrtzidis via cfe-commits
In r247832. > On Sep 12, 2015, at 5:19 AM, Manuel Klimek wrote: > > Test would be awesome :) Thx! > > On Fri, Sep 11, 2015 at 10:44 PM Argyrios Kyrtzidis > wrote: > In r247468, thanks for reviewing! > > >> On Sep 11, 2015, at 10:24 AM, Manuel Klimek via cfe-commit

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-12 Thread Manuel Klimek via cfe-commits
Test would be awesome :) Thx! On Fri, Sep 11, 2015 at 10:44 PM Argyrios Kyrtzidis wrote: > In r247468, thanks for reviewing! > > On Sep 11, 2015, at 10:24 AM, Manuel Klimek via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > > Ok, looked at the original patch again, and if we're fixing the

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-11 Thread Argyrios Kyrtzidis via cfe-commits
In r247468, thanks for reviewing! > On Sep 11, 2015, at 10:24 AM, Manuel Klimek via cfe-commits > wrote: > > Ok, looked at the original patch again, and if we're fixing the > FixedCompilationDatabase to only insert the file when it actually produces a > CompileCommand it seems to be fine. >

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-11 Thread Manuel Klimek via cfe-commits
Ok, looked at the original patch again, and if we're fixing the FixedCompilationDatabase to only insert the file when it actually produces a CompileCommand it seems to be fine. On Fri, Sep 11, 2015 at 6:32 PM Argyrios Kyrtzidis wrote: > On Sep 11, 2015, at 12:21 AM, Manuel Klimek wrote: > > On

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-11 Thread Argyrios Kyrtzidis via cfe-commits
> On Sep 11, 2015, at 12:21 AM, Manuel Klimek wrote: > > On Thu, Sep 10, 2015 at 8:36 PM Argyrios Kyrtzidis > wrote: >> On Sep 10, 2015, at 1:48 AM, Manuel Klimek > > wrote: >> >> @@ -179,11 +185,13 @@ public: >>/// \param Directory The

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-11 Thread Manuel Klimek via cfe-commits
On Thu, Sep 10, 2015 at 8:36 PM Argyrios Kyrtzidis wrote: > On Sep 10, 2015, at 1:48 AM, Manuel Klimek wrote: > > @@ -179,11 +185,13 @@ public: >/// \param Directory The base directory used in the > FixedCompilationDatabase. >static FixedCompilationDatabase *loadFromCommandLine(int &Argc

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-10 Thread Argyrios Kyrtzidis via cfe-commits
> On Sep 10, 2015, at 1:48 AM, Manuel Klimek wrote: > > @@ -179,11 +185,13 @@ public: >/// \param Directory The base directory used in the > FixedCompilationDatabase. >static FixedCompilationDatabase *loadFromCommandLine(int &Argc, >

Re: PATCH: Expose the 'file' that is associated with a compile database command

2015-09-10 Thread Manuel Klimek via cfe-commits
@@ -179,11 +185,13 @@ public: /// \param Directory The base directory used in the FixedCompilationDatabase. static FixedCompilationDatabase *loadFromCommandLine(int &Argc, const char *const *Argv, -

PATCH: Expose the 'file' that is associated with a compile database command

2015-09-09 Thread Argyrios Kyrtzidis via cfe-commits
Hi, The attached patch exposes the ‘file’ entry in a compilation database command, via the CompileCommand structure. compile_db_file.patch Description: Binary data ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/