[PATCH] D26224: NewGVN

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 11:13:21 PST 2016


davide added a comment.

In https://reviews.llvm.org/D26224#618046, @silvas wrote:

> Also, uhm, this needs tests. Maybe you can get away with just adding some more RUN lines to some LegacyGVN tests?


Yes, this is my intention. In the non-upstream branch all the tests have another run line which looks like
`RUN: opt -newgvn %s | FileCheck %s`. They're not included here just to make the diff smaller.


https://reviews.llvm.org/D26224





More information about the llvm-commits mailing list