<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 24, 2016 at 3:30 PM, Bruno Cardoso Lopes <span dir="ltr"><<a href="mailto:bruno.cardoso@gmail.com" target="_blank">bruno.cardoso@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Sure, go ahead.<br>
<br>
I committed in r284797 and r284801 (libcxx). There's one minor issue<br>
I've found: the changes for the builtins affecting non submodule local<br>
visibility broke current users of plain "-fmodules" against our<br>
frameworks in public SDKs, in 10.11 & 10.12. I've attached a patch to<br>
work around that for the time being: make the new behavior dependent<br>
on local vis. Can you take a look?</blockquote><div><br></div><div>What's the nature of the breakage? Generally I'd be fine with your patch, but I wonder if there's something better we could do here.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> Hmm. Ideally, we should try to pick something that captures the spirit of<br>
> "only non-modular headers and headers from used modules". Something like<br>
> "ignore_modules_not_declared_<wbr>used", but less wordy?<br>
<br>
</span>Right. It's gonna be hard to shrink this to a meaningful short name.<br>
What about a more generic "no_escape"?  "no_undeclared_headers"?<br></blockquote><div><br></div><div>Hmm. Maybe we could allow the existing [exhaustive] attribute to be specified on a use-declaration:</div><div><br></div><div>  use [exhaustive] a, b, c</div></div></div></div>