<div dir="ltr"><div>Michael</div><div><br></div>It appears your build fix and mine conflicted and caused another build break.<div>Ping me on IRC if you are planning to commit another fix.</div><div>I believe my latest commit fixes the last of the build breaks here.</div><div><br></div><div>Sorry about this. Have a nice day. </div><div><br></div><div>PL</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 30, 2019 at 12:22 PM Michael Liao via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Author: hliao<br>
Date: Fri Aug 30 12:23:28 2019<br>
New Revision: 370504<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=370504&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=370504&view=rev</a><br>
Log:<br>
Fix compilation warnings. NFC.<br>
<br>
Modified:<br>
    llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp<br>
<br>
Modified: llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp?rev=370504&r1=370503&r2=370504&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp?rev=370504&r1=370503&r2=370504&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp (original)<br>
+++ llvm/trunk/tools/llvm-ifs/llvm-ifs.cpp Fri Aug 30 12:23:28 2019<br>
@@ -35,7 +35,7 @@ using namespace llvm::MachO;<br>
<br>
 namespace {<br>
 const VersionTuple IFSVersionCurrent(1, 2);<br>
-};<br>
+}<br>
<br>
 static cl::opt<std::string> Action("action", cl::desc("<llvm-ifs action>"),<br>
                                    cl::value_desc("write-ifs | write-bin"),<br>
@@ -72,6 +72,7 @@ std::string getTypeName(IFSSymbolType Ty<br>
   case IFSSymbolType::Unknown:<br>
     return "Unknown";<br>
   }<br>
+  llvm_unreachable("Unexpected ifs symbol type.");<br>
 }<br>
<br>
 struct IFSSymbol {<br>
@@ -84,6 +85,8 @@ struct IFSSymbol {<br>
   bool operator<(const IFSSymbol &RHS) const { return Name < RHS.Name; }<br>
 };<br>
<br>
+namespace llvm {<br>
+namespace yaml {<br>
 /// YAML traits for IFSSymbolType.<br>
 template <> struct ScalarEnumerationTraits<IFSSymbolType> {<br>
   static void enumeration(IO &IO, IFSSymbolType &SymbolType) {<br>
@@ -151,6 +154,8 @@ template <> struct CustomMappingTraits<s<br>
       IO.mapRequired(Sym.Name.c_str(), const_cast<IFSSymbol &>(Sym));<br>
   }<br>
 };<br>
+} // End yaml namespace<br>
+} // End llvm namespace<br>
<br>
 // A cumulative representation of ELF stubs.<br>
 // Both textual and binary stubs will read into and write from this object.<br>
@@ -176,6 +181,8 @@ public:<br>
         Symbols(std::move(Stub.Symbols)) {}<br>
 };<br>
<br>
+namespace llvm {<br>
+namespace yaml {<br>
 /// YAML traits for IFSStub objects.<br>
 template <> struct MappingTraits<IFSStub> {<br>
   static void mapping(IO &IO, IFSStub &Stub) {<br>
@@ -189,6 +196,8 @@ template <> struct MappingTraits<IFSStub<br>
     IO.mapRequired("Symbols", Stub.Symbols);<br>
   }<br>
 };<br>
+} // End yaml namespace<br>
+} // End llvm namespace<br>
<br>
 static Expected<std::unique_ptr<IFSStub>> readInputFile(StringRef FilePath) {<br>
   // Read in file.<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>