
<!DOCTYPE html>

<html>

<head>

<title></title>

<style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style>

</head>

<body><div>That sounds quite reasaonable; how does one usually go about doing that? a repro zip that hits both asserts?<br></div>

<div><br></div>

<div><br></div>

<div>On Wed, Jul 11, 2018, at 03:11, Rui Ueyama wrote:<br></div>

<blockquote type="cite"><div dir="ltr"><div>Carlo,<br></div>

<div><br></div>

<div>Thank you for your contribution! We don't usually commit any feature without a test, so can you write a test for us? Then I think I can commit this for you.<br></div>

</div>

<div><br></div>

<div defang_data-gmailquote="yes"><div dir="ltr">On Tue, Jul 10, 2018 at 12:33 PM Andrew Kelley via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div>

<blockquote defang_data-gmailquote="yes" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div dir="ltr">I confirm that this passes the Zig test suite. This solves the 1 outstanding patch that Zig has against LLD, and if this is merged, we can go back to using LLD releases instead of a fork.<br></div>

<div><div><br></div>

<div defang_data-gmailquote="yes"><div>On Tue, Jul 10, 2018 at 3:40 AM, Carlo Kok via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>></span> wrote:<br></div>

<blockquote defang_data-gmailquote="yes" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div>On LLD mach/o x86_64 LLD calls dataInCodeTransitionStart and dataInCodeTransitionEnd, which aren't implemented (and thus return the defautl value 0), later in applyFixupFinal it reaches the default: unreachable causing miscompiles depending on the optimizer.<br></div>

<div> <br></div>

<div> Attached is a simple patch that fixes this. <br></div>

<div> <br></div>

<div> It also fixes the read8 method, as object files inside .a files aren't 8 byte aligned when memory mapped.<br></div>

<div> <br></div>

<div>_______________________________________________<br></div>

<div> LLVM Developers mailing list<br></div>

<div> <a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br></div>

<div> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></div>

<div> <br></div>

</blockquote></div>

<div><br></div>

</div>

<div>_______________________________________________<br></div>

<div> LLVM Developers mailing list<br></div>

<div> <a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br></div>

<div> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></div>

</blockquote></div>

</blockquote><div><br></div>

</body>

</html>