[PATCH] D12747: Implement [depr.c.headers]
Eric Fiselier via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 6 15:02:14 PDT 2015
LGTM.
On Tue, Oct 6, 2015 at 3:58 PM, Richard Smith <richard at metafoo.co.uk> wrote:
> On Mon, Oct 5, 2015 at 7:10 PM, Eric Fiselier <eric at efcs.ca> wrote:
>>
>> EricWF added a comment.
>>
>> I think thing change will help us close a number out outstanding bugs. I
>> don't have any fundamental objections to this approach. However the size of
>> this patch scares me. I understand the changes are mostly mechanical but
>> their size can hide things. For example has anybody noticed that your
>> internal changes to `<deque>` are in this patch? It would be nice to break
>> this down into more digestible pieces that could be quickly spot checked.
>
>
> OK. First such patch is attached. It just removes the macro-capturing
> wrapper functions.
More information about the cfe-commits
mailing list