================
@@ -8265,6 +8265,11 @@ def internal_externc_isystem : Separate<["-"], 
"internal-externc-isystem">,
            "implicit extern \"C\" semantics; these are assumed to not be "
            "user-provided and are used to model system and standard headers' "
            "paths.">;
+def internal_iframework : Separate<["-"], "internal-iframework">,
----------------
ldionne wrote:

I see. But there is no notion of "extern C" for framework includes, so IMO it 
makes more sense to make `-internal-iframework` "equivalent" to 
`-internal-isystem` than to `-internal-externc-isystem`, would you agree?

More generally, I guess I am confused about the desired grouping for framework 
includes. Do we want them to be `IncludeDirGroup::System`, 
`IncludeDirGroup::ExternCSystem` or something else? IIUC, `-iframework` is 
already part of `IncludeDirGroup::System`. If that's correct, then why is it 
not sufficient to pass these system framework paths using `-iframework` 
directly in the first place (i.e. why do we even need to introduce 
`-internal-iframework` in the first place)?

https://github.com/llvm/llvm-project/pull/120149
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to