<div dir="ltr">Are those tests actually invalid now? You can still stuff non-flag values into the flag with a static cast, so perhaps we should keep the test around in that form?</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 1, 2016 at 11:51 PM Victor <<a href="mailto:vleschuk@gmail.com">vleschuk@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vleschuk updated this revision to Diff 70135.<br>
vleschuk added a comment.<br>
<br>
Removed few deprecated tests which don't make sense with new type for flags.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23766" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23766</a><br>
<br>
Files:<br>
  include/llvm/IR/DIBuilder.h<br>
  include/llvm/IR/DebugInfoFlags.def<br>
  include/llvm/IR/DebugInfoMetadata.h<br>
  lib/AsmParser/LLParser.cpp<br>
  lib/Bitcode/Reader/BitcodeReader.cpp<br>
  lib/IR/AsmWriter.cpp<br>
  lib/IR/DIBuilder.cpp<br>
  lib/IR/DebugInfoMetadata.cpp<br>
  unittests/IR/DebugInfoTest.cpp<br>
  unittests/IR/DebugTypeODRUniquingTest.cpp<br>
  unittests/IR/IRBuilderTest.cpp<br>
  unittests/IR/MetadataTest.cpp<br>
  unittests/Transforms/Utils/Cloning.cpp<br>
<br>
</blockquote></div>