[PATCH] Fix frontend crash related to forward declaration of NSNumber

hfinkel at anl.gov hfinkel at anl.gov
Mon Feb 16 05:44:21 PST 2015


In http://reviews.llvm.org/D7643#124251, @AlexDenisov wrote:

> Not to give you a hard time, but, why are you asking for pre-commit review?
>  Well, due to first link (obtaining commit access) I got 'commit-after-approval’ access, so just trying to follow the rules.


Understood, but we do have this rule:

> 2. You are allowed to commit patches without approval which you think are obvious. This is clearly a subjective decision — we simply expect you to use good judgement. Examples include: fixing build breakage, reverting obviously broken patches, documentation/comment changes, any other minor changes.


and you're welcome to follow that one too ;)

> Anyway, thank you for feedback and useful links.

> 

> -- 

> 

> AlexDenisov

>  Software Engineer, https://github.com/AlexDenisov



http://reviews.llvm.org/D7643

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list