<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 13, 2014, at 3:35 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><blockquote type="cite"><br></blockquote><br>The whole file is like this. I can update all the functions in a follow<br>on patch.<br><br>The attached patch should address the rest of your comments.<br></div></blockquote></div><div><br></div><div>The new comment has a typo ‘consectuive’</div><div><br></div><div>AllocasToPromote looks unused</div><div><br></div><div><br></div><div>I also noticed FeatureLocalMemorySize65536 on CI. I thought it was still 32K? In my manual it still says 32K in section 2.3.1</div></body></html>