aprantl added a comment.

You are adding a new constant to the LLVM IR, so there should be a round-trip 
test that tests the LLVM IR parser, bitcode writer, bitcode reader, and LLVM IR 
printer to make sure we're handling the new constant correctly.

One easy way to do this is by adding the new flag to the already existing test 
in test/IR/disubprogram.ll.


https://reviews.llvm.org/D23167



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to