<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 14, 2016 at 9:59 AM Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br class="gmail_msg"><br>> On Oct 14, 2016, at 9:48 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="gmail_msg" target="_blank">dblaikie@gmail.com</a>> wrote:<br class="gmail_msg"><br>><br class="gmail_msg"><br>> Could someone point me to where the discussion for adding this type came out of? I didn't spot it at a cursory glance of the previous/existing threads.<br class="gmail_msg"><br>><br class="gmail_msg"><br><br class="gmail_msg"><br>This came out of the review thread in:<br class="gmail_msg"><br><br class="gmail_msg"><br>D25073: [DebugInfo]: preparation to implement DW_AT_alignment<br class="gmail_msg"><br><a href="https://reviews.llvm.org/D25073" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25073</a><br class="gmail_msg"><br><br class="gmail_msg"><br>where I argued that we should not be using a full uint64_t for the alignment fields in the DI.* metadata nodes. I don't think the concrete solution of introducing a new alignment type has been discussed here before.<br class="gmail_msg"></blockquote><div><br></div><div>Any particular benefits of introducing a new type? We don't I think have any wrapper types for otherwise singular numeric values, do we? (except maybe DIFlags?)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br class="gmail_msg"><br>-- adrian<br class="gmail_msg"><br></blockquote></div></div>