<div dir="ltr">Hi Tim,<div><br></div><div>I had a look and I'm afraid I haven't seen 0.1% of it. I don't think anyone will have time to review every line of it and not end up in a mental hospital, so I had a look at a few files, mostly the modified ones, and ignored the new ones.</div>
<div><br></div><div>Overall, the code seems fairly orthogonal to the current back-ends, and there's a large number of tests, which will help keeping it in shape in the near future.</div><div><div><br></div><div style>
A few questions:</div><div style><br></div><div style>Have you run the build and check-all on an ARM32 system? I say that because currently the ARM buildbots have at least three different types of failure, not to mention a "make clean" error, so adding a whole new back-end that has its own failures on ARM (especially because you probably develop on a 64-bit machine), won't help matters.</div>
<div style><br></div><div style>Now to code specific questions:</div><div style><br></div><div style><div>+// Vector Multiply-subtract (Floating Point)</div><div>+def int_aarch64_neon_vfms : Neon_3Arg_Intrinsic;</div><div>
(...)</div><div><div>+// Vector Multiply Extended (Floating Point)</div><div>+def int_aarch64_neon_vmulx : Neon_2Arg_Intrinsic;</div></div><div><br></div><div style>Can't these be patterns?</div><div style><br></div><div style>
<br></div><div style><div>-      APFloat apf(APInt::getNullValue(VT.getSizeInBits()));</div><div>+      APFloat apf(APInt::getNullValue(VT.getSizeInBits()), /*isIEEE=*/ true);</div><div><br></div><div style>I thought you had changed APFloat...</div>
<div style><br></div><div style><br></div><div style><div>-        ? dwarf::DW_EH_PE_udata4 : dwarf::DW_EH_PE_absptr;</div><div>+         ? dwarf::DW_EH_PE_udata4 : dwarf::DW_EH_PE_absptr;</div><div><br></div><div style>unnecessary space change, might confuse history</div>
<div style><br></div><div style><br></div><div style>That's all for now. I might have another look further down, but not sure that would actually help anything... ;)</div><div style><br></div><div style>cheers,</div><div style>
--renato</div></div></div></div></div></div>