<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 16, 2024 at 6:56 PM Mary Bennett <<a href="mailto:mary.bennett@embecosm.com" target="_blank">mary.bennett@embecosm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">v3 -> v4:<br>
 * Remove trailing white space<br>
 * Since the XCVbi instructions uses a relocation, linker tests were<br>
   added. I think it best if we keep them just to prove that the CORE-V<br>
   instructions are correct.<br></blockquote><div><br></div><div> I don't think keeping the ld testcase is necessary since you don't add any new relocation.  The cv.beqimm uses R_RISCV_BRANCH, and only changes the assembler code, so adding an assembler testcase with readelf or objdump -dr should be the right way to go.  Unless the cv.beqimm changes the R_RISCV_BRANCH behavior and need a new relocation, otherwise no ld testcase required.</div><div><br></div><div>Thanks</div><div>Nelson</div></div></div>