
<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">Oh you are right. Or users will get confused why their zext.h are changed to pack.<div>Thanks for reminding me that.</div><div><br></div><div><br><div><div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div>Hau Hsu</div><div><br></div><div><br></div></div><br class="Apple-interchange-newline"></div><br class="Apple-interchange-newline"><br class="Apple-interchange-newline">

</div>

<div><br><blockquote type="cite"><div>On Aug 5, 2024, at 2:17 PM, Nelson Chu <nelson@rivosinc.com> wrote:</div><br class="Apple-interchange-newline"><div><div dir="ltr"><div>Sorry that inverted - It means zext.h can use pack[w] encodings, but doesn't mean zext.h should be dumped as pack[w].</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 5, 2024 at 2:16 PM Nelson Chu <<a href="mailto:nelson@rivosinc.com">nelson@rivosinc.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 dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 5, 2024 at 12:07 PM Hau Hsu <<a href="mailto:hau.hsu@sifive.com" target="_blank">hau.hsu@sifive.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><div><div dir="auto" style="text-align:start;text-indent:0px"><div dir="auto" style="text-align:start;text-indent:0px"><div>> ... since pack[w] is a superset of zext.h, it makes sense for zext.h to use the pack[w] encoding in any situation that the full pack[w] instruction is provided (which for now I guess just means Zbkb).</div><div style="letter-spacing: normal; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;"><br></div><div style="letter-spacing: normal; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;">Thus we have this patch.</div><div style="letter-spacing: normal; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;"><br></div><div style="letter-spacing: normal; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;">And if this patch is feasible, I'll send v2 that follows Jan's suggestion to use alias instead of moving opcode table around.</div></div></div></div></div></div></blockquote><div><br></div><div>It means zext.h can use pack[w] encodings, but doesn't mean pack[w] should be dumped as zext.h.</div><div><br></div><div>Nelson</div><div> </div></div></div>

</blockquote></div></div>

</div></blockquote></div><br></div></body></html>