[PATCH] D34538: Handle ConstantExpr correctly in SelectionDAGBuilder

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 23 13:43:36 PDT 2017


efriedma added inline comments.


================
Comment at: test/CodeGen/Generic/pr33094.ll:2
+; RUN: llc < %s
+
+%A = type {}
----------------
RKSimon wrote:
> efriedma wrote:
> > Needs a comment describing what it's testing.
> > 
> > Writing tests without any CHECK lines is usually a bad idea, because they tend to be very easy to break, but I'm not sure what we would want to check for here, exactly, so I guess this is okay.
> Shouldn't this at least include a REQUIRES: asserts?
Why would we want "REQUIRES: asserts"?  The test isn't checking debug output, or anything like that.


https://reviews.llvm.org/D34538





More information about the llvm-commits mailing list