<div dir="ltr">That's not quite what I meant. I meant just have the places that need to know whether or not we're sanitizing look at the function attribute rather than resetting a value on MCOptions.<div><br></div><div>Thanks!</div><div><br></div><div>-eric<br><br><div class="gmail_quote">On Fri, May 8, 2015 at 4:16 PM Akira Hatanaka <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D9570#169482" target="_blank">http://reviews.llvm.org/D9570#169482</a>, @echristo wrote:<br>
<br>
> Hi Akira,<br>
><br>
> This is just moving the resetting to a different place. How about we get<br>
>  rid of the resetting completely?<br>
<br>
<br>
Hi Eric,<br>
<br>
Wouldn't getting rid of the resetting completely make it impossible to turn instrumentation on and off on a per-function basis? I don't know how important it is to have that capability, but it looks like r206971 was committed to enable it.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D9570" target="_blank">http://reviews.llvm.org/D9570</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div></div>