<div dir="ltr"><div><div>Jay, is this an answer to a question? If so, it's always best to leave the original question in the email for context.<br><br></div>Otherwise, is this a request for a fix/feature in compiler-rt?<br></div>If so, it is probably best to start with a PR on <a href="http://bugs.llvm.org">bugs.llvm.org</a> if it's a bug, a patch on <a href="http://reviews.llvm.org">reviews.llvm.org</a> if you're suggesting a change and/or an RFC on this list to explain your reasoning if you feel that this change may in some way be controversial or require input from the wider community.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Apr 15, 2018 at 9:23 AM, Jay K via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr">
<div id="m_-3349010807745490296divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif" dir="ltr">
<p style="margin-top:0;margin-bottom:0"></p>
<div>projects/compiler-rt/lib/<wbr>builtins/clear_cache.c</div>
<div><br>
</div>
<div>I suggest two small changes here.</div>
<div><br>
</div>
<div>1. Mark it however required to avoid inlining, i.e. for the x86/amd64 case.</div>
<div>Control transfer is required. I realize it is likely anyway, you aren't likely</div>
<div>to "fall from" the caller into the code.</div>
<div><br>
</div>
<div>2. #elif defined(_WIN32) && (defined(__arm__) || defined(__aarch64__))</div>
<div>be changed to just:</div>
<div>#elif defined(_WIN32)</div>
<div><br>
</div>
<div>since Win32 hypothetically/historically runs on more than just x86/amd64/arm/arm64, and</div>
<div>FlushInstructionCache has been and presumably will remain omnipresent.</div><span class="HOEnZb"><font color="#888888">
<div></div>
<div><br>
</div>
<div></div>
 - Jay
<p></p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<div id="m_-3349010807745490296Signature"><br>
</div>
</font></span></div>
</div>

<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>