I don’t think we really need this.  isn’t Ilya’s solution in the other patch already sufficient?<br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 7:34 AM Ivan Donchevskii 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">yvvan added a comment.<br>
<br>
@ilya-biryukov<br>
<br>
Hm. What about another way around? - We have user include paths (-I) and report them to the filesystem. This means that we have specific paths under which nothing can be mmaped and everything else can be. In particular cases we can also report -isystem there. This is quite the same logic as current isVolatile parameter but is set only once for each path.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D54995/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54995/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D54995" rel="noreferrer" target="_blank">https://reviews.llvm.org/D54995</a><br>
<br>
<br>
<br>
</blockquote></div>