
<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;">Wouldn't it be better if the tests did the "right thing" and called TheJIT->finalizeObject()?<div><br></div><div>I'm not sure how much I like SectionMemoryManager::applyPermissions() being specified to also mean flushing cache, but I don't have a strong opinion on that.</div><div><br></div><div>-Filip</div><div><br></div><div><br><div><div>On May 13, 2013, at 3:44 AM, David Tweed <<a href="mailto:david.tweed@arm.com">david.tweed@arm.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: 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;">Hi,<br><br>please find attached a minor clean-up patch for the MCJIT tests (along with<br>adding some explicit commentary to the applyPermissions()'s declarations).<br>Please review and if this is acceptable I'll commit to the codebase.<br><br>Cheers,<br>Dave<span><mcjit.diff></span></div></blockquote></div><br></div></body></html>