<div style="white-space:pre-wrap">No need to duplicate if you export the function or make it online anyway.<br><br>But having the comment makes sense. I'm ok with this.<br><br>Thank you,<br> Filipe</div><br><div class="gmail_quote"><div dir="ltr">On Tue, 29 Nov 2016 at 21:53, Kuba (Brecka) Mracek via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kubabrecka added a comment.<br class="gmail_msg">
<br class="gmail_msg">
> If that's correct, then can we use the same trick as DyldNeedsEnvVariable for ReexecDisabled? Or am I making these two to be the same issue when they aren't?<br class="gmail_msg">
<br class="gmail_msg">
Do you mean to check `dyldVersionNumber`?  That would just duplicate the code.  How about I just document this more in a comment and keep it as `return false` (for instrumented tests on macOS)?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D24699" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24699</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>