[PATCH] D66014: [analyzer] Avoid unnecessary enum range check on LValueToRValue casts
Chris Hamilton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 05:05:40 PDT 2019
chrish_ericsson_atx added a comment.
Kristoff, if you wouldn't mind, since you offered earlier, please go ahead and commit this change as-is, since it was accepted. I ran into some non-technical issues with my follow-up changes and I'm going to be unavailable for several weeks. To mitigate risk and work for my team, I'd like to submit the newer changes separately (and will reference this review in that changeset when I do, of course), after I return to work.
And-- thank you for your help and feedback! I appreciated this upstreaming process (especially for what seemed like a fairly small/simple chnage). I expect there will be many more.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66014/new/
https://reviews.llvm.org/D66014
More information about the cfe-commits
mailing list