<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=windows-1257">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
I'm the one who has been working on this patch.</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
I have noticed the test failures, but I cannot reproduce those when I build and test locally, so I haven't actively tried to do anything about them. I don't put much stock in the pre-land tests, they tend to fail in odd ways.<br>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> llvm-dev <llvm-dev-bounces@lists.llvm.org> on behalf of David Lloyd via llvm-dev <llvm-dev@lists.llvm.org><br>
<b>Sent:</b> Friday, September 4, 2020 4:42 PM<br>
<b>To:</b> llvm-dev@lists.llvm.org <llvm-dev@lists.llvm.org><br>
<b>Subject:</b> Re: [llvm-dev] Saturating float-to-int casts</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">On Fri, Aug 7, 2020 at 9:19 AM David Lloyd <david.lloyd@redhat.com> wrote:<br>
><br>
> I have encountered a need for float-to-int casts that saturate to<br>
> min/max when the value is out of the range of the target type. It<br>
> seems that there is no intrinsic to do this, currently, but on IRC it<br>
> was pointed out that a patch [1] has been proposed to implement this<br>
> functionality in exactly the way that I was looking for.<br>
><br>
> It looks like the discussion has died out but I was hoping maybe to<br>
> kick it off again.<br>
><br>
> [1] <a href="https://protect2.fireeye.com/v1/url?k=0f2670d1-5186b297-0f26304a-86fc6812c361-8897797f7301e1e5&q=1&e=215b56a1-2b93-4622-a733-7123494b2246&u=https%3A%2F%2Freviews.llvm.org%2FD54749">
https://protect2.fireeye.com/v1/url?k=0f2670d1-5186b297-0f26304a-86fc6812c361-8897797f7301e1e5&q=1&e=215b56a1-2b93-4622-a733-7123494b2246&u=https%3A%2F%2Freviews.llvm.org%2FD54749</a><br>
<br>
Looks like a little more activity has happened, and the patch was<br>
rebased, but some unit tests are currently failing (I have no idea how<br>
normal that is but I'm assuming it's a problem - at least the failures<br>
seem related, on the surface anyway). Seems like the failures are<br>
related to applying these operations to vectors.<br>
<br>
-- <br>
- DML • he/him<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
llvm-dev@lists.llvm.org<br>
<a href="https://protect2.fireeye.com/v1/url?k=898c1b6b-d72cd92d-898c5bf0-86fc6812c361-8e5301a59ca3fac6&q=1&e=215b56a1-2b93-4622-a733-7123494b2246&u=https%3A%2F%2Flists.llvm.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fllvm-dev">https://protect2.fireeye.com/v1/url?k=898c1b6b-d72cd92d-898c5bf0-86fc6812c361-8e5301a59ca3fac6&q=1&e=215b56a1-2b93-4622-a733-7123494b2246&u=https%3A%2F%2Flists.llvm.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fllvm-dev</a><br>
</div>
</span></font></div>
</body>
</html>