[PATCH] D52002: Omit range checks from jump tables when lowering switches with unreachable default

Ayonam Ray via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 21 06:41:53 PST 2018


ayonam added a comment.

In D52002#1339026 <https://reviews.llvm.org/D52002#1339026>, @hans wrote:

> Looks good to me. Do you have commit rights, or do you need someone to commit it for you?
>
> (The change description is still a little out of date, but that can be fixed when committing, if not before.)


Thanks Hans for all your efforts.  I have updated the description.  Feel free to edit it if you think it can be made more concise.

I have never committed before.  So I do not have commit rights.  It would be great if you could kindly commit the patch for me.

Also, could you please review the patch D52707 <https://reviews.llvm.org/D52707> for a commit?  I have made the changes you mentioned before.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D52002/new/

https://reviews.llvm.org/D52002





More information about the llvm-commits mailing list