<div dir="ltr"><div dir="ltr"><div dir="ltr">There are more warnings.<div><a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/15769/steps/bootstrap%20clang/logs/stdio">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/15769/steps/bootstrap%20clang/logs/stdio</a><pre style="font-family:"Courier New",courier,monotype,monospace;color:rgb(0,0,0);font-size:medium;text-decoration-style:initial;text-decoration-color:initial"><span class="gmail-stdout" style="font-family:"Courier New",courier,monotype,monospace;color:black">/b/sanitizer-x86_64-linux/build/llvm/lib/DebugInfo/PDB/PDBExtras.cpp:68:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default]
  default:</span></pre><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 20, 2018 at 10:48 AM, Zachary Turner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: zturner<br>
Date: Thu Sep 20 10:48:44 2018<br>
New Revision: 342670<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=342670&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=342670&view=rev</a><br>
Log:<br>
Fix warnings.<br>
<br>
Modified:<br>
    llvm/trunk/include/llvm/<wbr>DebugInfo/PDB/Native/<wbr>TpiHashing.h<br>
    llvm/trunk/lib/DebugInfo/PDB/<wbr>Native/TpiStream.cpp<br>
<br>
Modified: llvm/trunk/include/llvm/<wbr>DebugInfo/PDB/Native/<wbr>TpiHashing.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/DebugInfo/PDB/Native/TpiHashing.h?rev=342670&r1=342669&r2=342670&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/llvm/trunk/include/<wbr>llvm/DebugInfo/PDB/Native/<wbr>TpiHashing.h?rev=342670&r1=<wbr>342669&r2=342670&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- llvm/trunk/include/llvm/<wbr>DebugInfo/PDB/Native/<wbr>TpiHashing.h (original)<br>
+++ llvm/trunk/include/llvm/<wbr>DebugInfo/PDB/Native/<wbr>TpiHashing.h Thu Sep 20 10:48:44 2018<br>
@@ -21,19 +21,19 @@ Expected<uint32_t> hashTypeRecord(const<br>
 struct TagRecordHash {<br>
   explicit TagRecordHash(codeview::<wbr>ClassRecord CR, uint32_t Full,<br>
                          uint32_t Forward)<br>
-      : Class(std::move(CR)), FullRecordHash(Full), ForwardDeclHash(Forward) {<br>
+      : FullRecordHash(Full), ForwardDeclHash(Forward), Class(std::move(CR)) {<br>
     State = 0;<br>
   }<br>
<br>
   explicit TagRecordHash(codeview::<wbr>EnumRecord ER, uint32_t Full,<br>
                          uint32_t Forward)<br>
-      : Enum(std::move(ER)), FullRecordHash(Full), ForwardDeclHash(Forward) {<br>
+      : FullRecordHash(Full), ForwardDeclHash(Forward), Enum(std::move(ER)) {<br>
     State = 1;<br>
   }<br>
<br>
   explicit TagRecordHash(codeview::<wbr>UnionRecord UR, uint32_t Full,<br>
                          uint32_t Forward)<br>
-      : Union(std::move(UR)), FullRecordHash(Full), ForwardDeclHash(Forward) {<br>
+      : FullRecordHash(Full), ForwardDeclHash(Forward), Union(std::move(UR)) {<br>
     State = 2;<br>
   }<br>
<br>
<br>
Modified: llvm/trunk/lib/DebugInfo/PDB/<wbr>Native/TpiStream.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/PDB/Native/TpiStream.cpp?rev=342670&r1=342669&r2=342670&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/llvm/trunk/lib/<wbr>DebugInfo/PDB/Native/<wbr>TpiStream.cpp?rev=342670&r1=<wbr>342669&r2=342670&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- llvm/trunk/lib/DebugInfo/PDB/<wbr>Native/TpiStream.cpp (original)<br>
+++ llvm/trunk/lib/DebugInfo/PDB/<wbr>Native/TpiStream.cpp Thu Sep 20 10:48:44 2018<br>
@@ -200,7 +200,6 @@ TpiStream::<wbr>findFullDeclForForwardRef(Typ<br>
   if (!ForwardTRH)<br>
     return ForwardTRH.takeError();<br>
<br>
-  TagRecordHash Copy = std::move(*ForwardTRH);<br>
   uint32_t BucketIdx = ForwardTRH->FullRecordHash % Header->NumHashBuckets;<br>
<br>
   for (TypeIndex TI : HashMap[BucketIdx]) {<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>