[PATCH] D21775: [InstCombine] Simplify and correct folding fcmps with the same children
Eric Christopher via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 28 21:35:10 PDT 2016
Sounds good. Thanks for checking.
On Tue, Jun 28, 2016, 9:26 PM Tim Shen <timshen at google.com> wrote:
> On Tue, Jun 28, 2016 at 7:32 PM Eric Christopher <echristo at gmail.com>
>> My guess is just automatic naming convention changes as part of the
>> script run. Try running it on the file without a change in tree to update
>> it, take a look at that and possibly minimize, and then once after your
> Running it on the clean top of the tree produces no diffs. :)
> The name update_test_checks generated depends on the variable name
> produced by, in our case, opt:
> As my patch goes in, the value names opt produce indeed change, which
> cause the CHECK variable names' change. I'm fine with the name changes as
> they are not stable anyway.
> We could change update_test_checks to always use its own names and
> counter, but that's another topic.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits