Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Patch for review

Thanks Davin for the ping. Sorry that such pinging is sometimes necessary to bring things to the top of our lists again, but I have now reviewed your change and thank you for providing it.

For Davin and everyone else, the pinging of code reviews is part of CDT's process, so don't feel bad doing the pinging. Ideally some feedback would go to people as soon as possible, but sometimes things come up. I (and I hope all the other CDT committers) will continue to strive to continuously be better in this regard.

If it has not received attention in a week or so, some nagging can help. Send email to mailto:cdt-dev@xxxxxxxxxxx asking committers to look at the contribution. Continue sending e-mails until somebody would give up and look :)


Jonah Graham
Kichwa Coders

On Fri, 24 Jun 2022 at 18:52, Davin McCall <davmac@xxxxxxxxxx> wrote:

Hi all,

Would anyone be able to review my patch, here:

This is for bug 580201, "constexpr evaluation of builtins doesn't narrow integer arguments", and builds on the work I did earlier to support constexpr evaluation of various builtins for the last release.



cdt-dev mailing list
To unsubscribe from this list, visit

Back to the top