
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

</head>

<body>

Hi,<br>

<br>

How can we make InstSimplify/InstCombine work with intrinsics?<br>

<br>

This RFC proposes generalized pattern matching to lift LLVM's existing InstSimplify to work on intrinsics as well as on regular instructions. For details, technical discussion, please refer to:<br>

<br>

    <a class="moz-txt-link-freetext" href="https://reviews.llvm.org/D92086">https://reviews.llvm.org/D92086</a><br>

<p>The patch includes tests where InstSimplify optimizes @llvm.vp.add and @llvm.experimental.contrained.fadd as a proof of concept.<br>

</p>

<p>The idea is to pretend that the intrinsics were the actual instruction (eg an <tt class="remarkup-monospaced">

fadd</tt>) and run the existing InstSimplify logic. We augment pattern matching with a context object - if during pattern matching the context object is no longer sure that the intrinsics are a safe match (eg because there may be fp exceptions that we need

 to preserve), it aborts the match. In effect, only legal simplifications are performed.</p>

<p>We expect the approach to work as well for the [proposed] complex arithmetic intrinsics (by masking regular `fmul` and making `llvm.complex.fmul` pass as `fmul` instead).</p>

<p>The current patch only considers InstSimplify where patterns are replaced by leaf values. We know how to extend this to InstCombine where new code is emitted to replace the old one (by passing the context object to an IR Builder class) - this was implemented

 before in <a class="moz-txt-link-freetext" href="https://reviews.llvm.org/D57504">

https://reviews.llvm.org/D57504</a> for VP intrinsics.<br>

</p>

- Simon<br>

</body>

</html>