<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 5, 2016 at 3:59 PM Greg Clayton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg updated this revision to Diff 80340.<br class="gmail_msg">
clayborg added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Fixed Adrian's issues. Big changes include:<br class="gmail_msg">
<br class="gmail_msg">
- Added llvm::Expected support to the dwarfgen::Generator<br class="gmail_msg">
- Handle errors if any arise<br class="gmail_msg">
- Get host triple and get its 32 and 64 bits variants for 4 byte and 8 byte addresses and bail out if targets are not available without breaking the gtest.<br class="gmail_msg"></blockquote><div><br></div><div>It's possible to build the compiler without support for the host architecture, I should think... <br><br>It's probably simpler/OK (open to debate/disagreement, for sure) hardcode some architecture and use lit test config to opt out of running these tests unless that target is built in. (lots of our lit tests do this because they test some specific architecture feature - not sure if any of the unit tests do, though, os that might be a bit trickier)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D27326" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D27326</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
  include/llvm/CodeGen/DIE.h<br class="gmail_msg">
  include/llvm/CodeGen/DIEValue.def<br class="gmail_msg">
  include/llvm/DebugInfo/DWARF/DWARFDebugInfoEntry.h<br class="gmail_msg">
  include/llvm/MC/MCContext.h<br class="gmail_msg">
  lib/CodeGen/AsmPrinter/DIE.cpp<br class="gmail_msg">
  lib/CodeGen/AsmPrinter/DIEHash.cpp<br class="gmail_msg">
  lib/CodeGen/AsmPrinter/DwarfFile.cpp<br class="gmail_msg">
  lib/CodeGen/AsmPrinter/DwarfFile.h<br class="gmail_msg">
  lib/CodeGen/CMakeLists.txt<br class="gmail_msg">
  lib/CodeGen/DwarfGenerator.cpp<br class="gmail_msg">
  lib/CodeGen/DwarfGenerator.h<br class="gmail_msg">
  lib/DebugInfo/DWARF/DWARFDebugInfoEntry.cpp<br class="gmail_msg">
  unittests/DebugInfo/DWARF/CMakeLists.txt<br class="gmail_msg">
  unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>