<div dir="ltr"><div>Thanks - I was able to get past this problem.</div><div><br></div><div>For the benefit of anyone googling a similar problem after me:</div><div><br></div><div>I had to fix two separate problems:</div><div><br></div><div>1. Because intrinsic enum definitions are split out into target-specific includes, I needed to name my intrinsics properly.</div><div><br></div><div>The target splitter assumes that intrinsics of the form "llvm.foo..." go into "IntrinsicsFoo.h". For this to work, "foo" must be a known target.</div><div><br></div><div>In my case, I was extending the RISCV instruction set with an intrinsic named "int_idaho_mt_begin".  Since "idaho" is not a valid target, I needed to rename to "int_riscv_idaho_mt_begin". Note that doing so may require changes to both include/llvm/IR/IntrinsicsXXX.h and lib/Target/XXX/XXXInstrInfo.td (or some file referenced by it).  You will also have to update any front ends to use the new names.</div><div><br></div><div>2. <span style="font-family:monospace">RISCVGenGlobalISel.inc<font face="arial,sans-serif"> is included from </font>RISCVInstructionSelector.cpp.<font face="arial,sans-serif"> Since this file does not already reference intrinsic definitions, the include needed to be added there. The full path relative to your build directory is required:</font></span></div><div><span style="font-family:monospace"><font face="arial,sans-serif"><br></font></span></div><div><span style="font-family:monospace"><font face="arial,sans-serif"><span style="font-family:monospace">#include "llvm/IR/IntrinsicsRISCV.h"</span><br></font></span></div><div><span style="font-family:monospace"><br></span></div><div><span style="font-family:monospace"><font face="arial,sans-serif">With these changes I was able to generate code using my intrinsics with LLVM 10.</font><br></span></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 6, 2020 at 2:19 PM Krzysztof Parzyszek <<a href="mailto:kparzysz@quicinc.com" target="_blank">kparzysz@quicinc.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-US">
<div>
<p class="MsoNormal">Intrinsic headers have been broken up into smaller files.  Now you need to include the target-specific intrinsic header, e.g. IntrinsicsRISCV.h.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><span style="font-size:9pt;font-family:Consolas">-- </span>
<span style="font-size:9pt;font-family:Consolas"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:8pt;font-family:Consolas">Krzysztof Parzyszek 
<a href="mailto:kparzysz@quicinc.com" target="_blank"><span style="color:rgb(5,99,193)">kparzysz@quicinc.com</span></a>   AI tools development<u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-color:currentcolor currentcolor currentcolor blue;border-style:none none none solid;border-width:medium medium medium 1.5pt;padding:0in 0in 0in 4pt">
<div>
<div style="border-color:rgb(225,225,225) currentcolor currentcolor;border-style:solid none none;border-width:1pt medium medium;padding:3pt 0in 0in">
<p class="MsoNormal"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> <b>On Behalf Of
</b>David Jones via llvm-dev<br>
<b>Sent:</b> Wednesday, May 6, 2020 1:08 PM<br>
<b>To:</b> llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>><br>
<b>Subject:</b> [EXT] [llvm-dev] Issues porting intrinsics to LLVM 10<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">I am maintaining proprietary extensions to the RISCV backend for our custom application.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I have defined intrinsics for many of the custom instructions.  Against LLVM 7 this was working well.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">When I try to merge my changes into LLVM 10, I get:<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Courier New"">/home/dej/work/llvm_git/llvm-project/llvm/build/lib/Target/RISCV/RISCVGenGlobalISel.inc:11582:60: error: ‘idaho_mt_begin’ is not a member of ‘llvm::Intrinsic’<br>
         GIM_CheckIntrinsicID, /*MI*/0, /*Op*/0, Intrinsic::idaho_mt_begin,</span><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">This intrinsic is defined pretty simply:<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Courier New"">let TargetPrefix = "idaho" in {<br>
def int_idaho_mt_begin : Intrinsic<[], [llvm_ptr_ty,llvm_i64_ty], []>;</span><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><span style="font-family:"Courier New"">}</span><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">With LLVM 7, I notice that build/include/llvm/IntrinsicEnums.inc contains an entry for each intrinsic in the system.  This is no longer the case for LLVM 10: there are only 277 definitions in the file. It seems that LLVM 10 imposes some
 filter condition where it deems inclusion in the IntrinsicEnums.inc unnecessary.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">The other thing I notice: GlobalISel was not built (by default) in LLVM 7, but it is for LLVM 10.  This is the entity referencing the enums.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I tried renaming my intrinsics to replace "idaho" with "riscv" speculating that this was the filtering condition, but that did not make any difference.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">What has changed?<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div>
</div>
</div>

</blockquote></div>