<div dir="ltr">hmm... thinking about this more we don't actually need this condition<br><br>      else if (sizeof(long)==8 && bytes==8) data = *(unsigned long *)p;<br><br><div>because we already check that we don't let you set bytes to 8 if you're on ILP32 (which is good, because _that_ would be a bug).</div><div><br></div><div>so we can just write<br><br>      else data = *(unsigned long *)p;<br><br></div><div>here (and in the other "copy" of this if above) and make this go away with less code? if we do that, we can remove the QUIET from data completely (because at least with `make defconfig` gcc 13 doesn't complain about this code at all).</div><div><br></div><div>and if we do _that_, then i wonder whether we should keep clang on the "no-op QUIET" path until/unless we really do need it, to force us to continue to look at and think about these things? there are very few uses of QUIET, and it does seem like something we should avoid...</div><div><br></div><div>patch attached...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 7, 2024 at 10:28 AM enh <<a href="mailto:enh@google.com">enh@google.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"><br>---<br> lib/portability.h | 5 +----<br> 1 file changed, 1 insertion(+), 4 deletions(-)<br><br></div>
</blockquote></div>