Re: [PATCH] Add -finstruments-functions-exclude-{file, function}-list options

2013-11-19 Thread Gonzalo BG
gnzlbg added you to the CC list for the revision Add -finstruments-functions-exclude-{file, function}-list options. This options allow to exclude functions from instrumentation either by providing a list of user-visible function names or a list of file paths. These allows to exclude e.g. the

Re: [PATCH] Add -finstruments-functions-exclude-{file, function}-list options

2013-03-13 Thread Douglas Gregor
On Mar 11, 2013, at 9:31 PM, Matthew Iselin matt...@theiselins.net wrote: Hi all, This patch adds two options, -finstrument-functions-exclude-file-list= and -finstrument-functions-exclude-function=list=, which offer a means through which headers in various locations and individual

Re: [PATCH] Add -finstruments-functions-exclude-{file, function}-list options

2013-03-13 Thread Dmitri Gribenko
On Tue, Mar 12, 2013 at 6:31 AM, Matthew Iselin matt...@theiselins.net wrote: This patch adds two options, -finstrument-functions-exclude-file-list= and -finstrument-functions-exclude-function=list=, which offer a means through which headers in various locations and individual functions can

Re: [PATCH] Add -finstruments-functions-exclude-{file, function}-list options

2013-03-13 Thread Matthew Iselin
Hi, I have attached a new patch that only fixes the inline issue. On 14/03/2013, at 9:06 AM, Douglas Gregor dgre...@apple.com wrote: ... I suggest that we look through macro expansions to find the FileID where the expansion occurred, then check that. Also, can we cache the result of this

[PATCH] Add -finstruments-functions-exclude-{file, function}-list options

2013-03-11 Thread Matthew Iselin
Hi all, This patch adds two options, -finstrument-functions-exclude-file-list= and -finstrument-functions-exclude-function=list=, which offer a means through which headers in various locations and individual functions can be excluded from instrumentation via -finstrument-functions. These