<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Dale,<div><br></div><div>Future patches are best sent to llvm-commits, or posted on Phabricator (<a href="https://llvm.org/docs/Phabricator.html" target="_blank">https://llvm.org/docs/Phabricator.html</a>).</div><div><br></div><div>This is a good fix though -- I've committed it in b45b5166f8f. Thanks for the contribution!</div><div><br></div><div>Regards,</div><div>Lang.</div><div><br></div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 6, 2020 at 1:06 PM Dale Martin via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div>
<p class="MsoNormal">Here’s a 2 line patch I would like to propose:<u></u><u></u></p>
<ol style="margin-top:0in" start="1" type="1">
<li style="margin-left:0in">Fix a typo s/anordered/an ordered/g<u></u><u></u></li><li style="margin-left:0in">Instead of just printing to llvm::errs, report a fatal error and stop.<u></u><u></u></li></ol>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">If there is a better list to discuss this on, please let me know.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
<p class="MsoNormal">  Dale<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">diff --git a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldCOFFX86_64.h b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldCOFFX86_64.h<u></u><u></u></p>
<p class="MsoNormal">index aee5f6dc374..f34c93e8007 100644<u></u><u></u></p>
<p class="MsoNormal">--- a/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldCOFFX86_64.h<u></u><u></u></p>
<p class="MsoNormal">+++ b/llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldCOFFX86_64.h<u></u><u></u></p>
<p class="MsoNormal">@@ -114,8 +114,7 @@ public:<u></u><u></u></p>
<p class="MsoNormal">       // memory layout to be: CodeSection < ReadOnlySection < ReadWriteSection.<u></u><u></u></p>
<p class="MsoNormal">       const uint64_t ImageBase = getImageBase();<u></u><u></u></p>
<p class="MsoNormal">       if (Value < ImageBase || ((Value - ImageBase) > UINT32_MAX)) {<u></u><u></u></p>
<p class="MsoNormal">-        llvm::errs() << "IMAGE_REL_AMD64_ADDR32NB relocation requires an"<u></u><u></u></p>
<p class="MsoNormal">-                     << "ordered section layout.\n";<u></u><u></u></p>
<p class="MsoNormal">+        report_fatal_error("IMAGE_REL_AMD64_ADDR32NB relocation requires an ordered section layout");<u></u><u></u></p>
<p class="MsoNormal">         write32BitOffset(Target, 0, 0);<u></u><u></u></p>
<p class="MsoNormal">       } else {<u></u><u></u></p>
<p class="MsoNormal">         write32BitOffset(Target, RE.Addend, Value - ImageBase);<u></u><u></u></p>
</div>
</div>

_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>