[PATCH] D66014: [analyzer] Avoid unnecessary enum range check on LValueToRValue casts
Chris Hamilton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 15 09:30:44 PDT 2019
chrish_ericsson_atx marked 5 inline comments as done.
chrish_ericsson_atx added a comment.
In D66014#1627858 <https://reviews.llvm.org/D66014#1627858>, @Szelethus wrote:
> LGTM, thanks! Do you need someone to commit this on your behalf? Also, could you please make the comments capitalized, terminated, and fitting in 80 columns?
I have updated the comments and line formatting as you recommended. Given that this change is already "Accepted", can I (should I) upload new differential for this change, or should this be delivered as-is, and I'll upload the new diffs as a new/separate change?
And yes, I will need someone to commit on my behalf. I'm too new to have commit privs. :)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66014/new/
https://reviews.llvm.org/D66014
More information about the cfe-commits
mailing list