<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 24, 2020 at 12:29 PM James Y Knight <<a href="mailto:jyknight@google.com">jyknight@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">Making a change like that privately while you're working through the list of callers could be a useful strategy to help keep track of which callers you've checked for correctness already. It might even be useful to send in a draft review, to ask for help in determining which of these callsites should and should not be modified.<div><br></div><div>But I don't think it'd be useful to actually check that change in. </div></div></blockquote><div>Actually checking it in temporarily can locate additional calls in patches-in-flight. That is a possible advantage.<br></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><br></div><div>I currently would expect that the vast majority of the time, the code is correct to be using the guaranteed "ABI" alignment, and only a small number of places need to be changed to use the preferred type alignment.<br></div></div></blockquote><div>Are these places sufficiently rare that we are unconcerned with occurrences of these that may be exclusively downstream at this point in time?<br></div></div></div>