[PATCH] SDAG: Update ChainNodesMatched during UpdateChains if a node is replaced

Ulrich Weigand via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 05:33:05 PST 2017


Justin Bogner <justin at justinbogner.com> wrote on 28.01.2017 01:38:23:

> Fix for llvm.org/PR31710. I'm not sure I love the test case (it's just a
> crash test) but I couldn't come up with anything more specific.

Yes, this is really hard to trigger, I had a hard time coming up with
any better test case as well ...

> Does this look good to commit?

LGTM.

Thanks,
Ulrich
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170130/cdabf325/attachment.html>


More information about the llvm-commits mailing list