Re: [PATH] Patch to fix -finstrument-functions-exclude-function-list handling of namespaces and escaped commas

2019-07-24 Thread Jeff Law
On 7/4/19 2:18 AM, Oliver Browne wrote: > See below for modified patch, indentation and newline for curly braces > style applied, and commented out chunk removed. Apologies, indentation > and newline for scope are not they way I normally write things, habits > got the better of me, and I forgot to

Re: [PATH] Patch to fix -finstrument-functions-exclude-function-list handling of namespaces and escaped commas

2019-07-04 Thread Oliver Browne
See below for modified patch, indentation and newline for curly braces style applied, and commented out chunk removed. Apologies, indentation and newline for scope are not they way I normally write things, habits got the better of me, and I forgot to remove the commented out chunk before

Re: [PATH] Patch to fix -finstrument-functions-exclude-function-list handling of namespaces and escaped commas

2019-07-03 Thread Jeff Law
On 6/12/19 12:25 PM, Oliver Browne wrote: > Patch fixes following PRs: > c++/90816 - -finstrument-functions-exclude-function-list improperly > handles namespace/class definitions > c++/90809 - -finstrument-functions-exclude-function-list mishandles > comma escaping > > Fixes as follows: > At

[PATH] Patch to fix -finstrument-functions-exclude-function-list handling of namespaces and escaped commas

2019-06-12 Thread Oliver Browne
Patch fixes following PRs: c++/90816 - -finstrument-functions-exclude-function-list improperly handles namespace/class definitions c++/90809 - -finstrument-functions-exclude-function-list mishandles comma escaping Fixes as follows: At flag_instrument_functions_exclude_p [gimplify.c] Using