<div dir="ltr"><div dir="ltr">On Thu, 4 Feb 2021 at 10:29, Adalid Claure via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><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 style="font-size:small">Hello,</div><div style="font-size:small"><br></div><div style="font-size:small">I was inspired by a recent tweet that said the C++23 feature of adding size_t suffixes was checked into gcc. I have never worked on the clang code base, much less tried to implement a feature, but I figured since the scope of this feature was relatively small, it might be a good place to start.</div><div style="font-size:small"><br></div><div style="font-size:small">Attached is my initial patch.</div><div style="font-size:small"><br></div><div style="font-size:small">I am looking for some guidance as to what kind of unit tests and other considerations would be required before this patch could be accepted.</div><div style="font-size:small"><br></div><div style="font-size:small">Any feedback would be greatly appreciated.</div></div></blockquote><div><br></div><div>'size_t' isn't necessarily 'unsigned long'; I think we'll need a flag separate from 'isLong' to track whether 'z' was present.</div><div> </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 style="font-size:small">Thank you!</div><div style="font-size:small"><br></div><div style="font-size:small"><br></div><div style="font-size:small"></div><div style="font-size:small"><br></div></div>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>