[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