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

hfinkel at anl.gov hfinkel at anl.gov
Mon Feb 16 05:35:04 PST 2015


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

> Ping?
>
> It’s a trivial one, but I don’t want to push it without review, though it fixes crash


Not to give you a hard time, but, why are you asking for pre-commit review? It sounds like you consider this to be a fairly-obvious one-line change to fix a crash-on-invalid case (and that is indeed what it looks like). If so, you don't need pre-commit review, so please feel free to commit. If you'd like a pre-commit review anyway, it is helpful to explain your doubts so potential reviews have some idea about what you're unsure, and are aware of any potential complications you're contemplating.

See: http://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access

Also, for the future, please upload full-context patches to Phabricator, see: http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface

> -- 

> 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