<div dir="ltr">Hi All,<div><br></div><div>With the change to use target attributes in the builtin headers, we are now in a weird situation where all of the `requires` inside lib/Headers/module.modulemap are basically useless. They are included regardless of the status of their `requires`. My pending patch <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10423&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=uocMWyh0ReeAMy12DGAd3JLFRdQVyPyhHC2qvGE_XIc&s=Oq-3NmxbBMWIl0eTMcbnqa3fytqHaXYul86FOLRPNUY&e=">http://reviews.llvm.org/D10423</a> then causes a build failure because we try to import unavailable modules; without it, we just textually include them which causes the declarations to appear to be owned by different modules than expected.</div><div><br></div><div>Any ideas? It seems like to keep semantic parity with the "all intrinsic headers can be brought in" when textually including, we should remove the `requires`.</div><div><br></div><div>-- Sean Silva</div></div>