{"push_id":3099564769,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"0ea51136ebeb9c4dc7facd0fbb6fc2f6da0e31e7","before":"fa434167fa2ebf575f489a9c893b986bb3e7451b","commits":[{"sha":"9b0a5d546988831ce3777cf9dc101e22bd44c041","author":{"name":"Sanjay Patel","email":"4a8dfeb3a1c6b90c15fd5a5a91e8e25ea46ffe0b@rotateright.com"},"message":"[DAGCombiner] guard against an oversized shift crash\n\nThis change prevents the crash noted in the post-commit comments \nfor rL347478 :\nhttp://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20181119/605166.html\n\nWe can't guarantee that an oversized shift amount is folded away, \nso we have to check for it.\n\nNote that I committed an incomplete fix for that crash with:\nrL347502\n\nBut as discussed here:\nhttp://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20181126/605679.html\n...we have to try harder.\n\nSo I'm not sure how to expose the bug now (and apparently no fuzzers have found \na way yet either).\n\nOn the plus side, we have discovered that we're missing real optimizations by \nnot simplifying nodes sooner, so the earlier fix still has value, and there's \nlikely more value in extending that so we can simplify more opcodes and simplify \nwhen doing RAUW and/or putting nodes on the combiner worklist.\n\nDifferential Revision: https://reviews.llvm.org/D54954\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@348089 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/9b0a5d546988831ce3777cf9dc101e22bd44c041"},{"sha":"0ea51136ebeb9c4dc7facd0fbb6fc2f6da0e31e7","author":{"name":"LLVM Monorepo conversion","email":"cd21cd76f101be56d104584fc5a716fe3b563fb7@llvm.org"},"message":"Merging r348089: \"[DAGCombiner] guard against an oversized shift crash\"\n\ngit-crossrepo-prefix: llvm/","distinct":true,"url":"https://api.github.com/repos/bogner/llvm-zipper-prototype/commits/0ea51136ebeb9c4dc7facd0fbb6fc2f6da0e31e7"}]}
{"actor":{"display_login":"bogner"}}