[PATCH] D47026: [DAG] fold FP binops with undef operands to NaN

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 17 12:58:21 PDT 2018


jlebar added a comment.

> NVPTX/implicit-def.ll - this isn't testing what it intended to test, but I don't know how to do that...could just delete the file?

This test is from 2013 and hasn't had a meaningful change since then.  It's not totally clear to me what it's testing, but it looks like it's checking for a crasher.

I suspect that in the years since then we're a-ok on covering this.  I support deleting it.


https://reviews.llvm.org/D47026





More information about the llvm-commits mailing list