[cfe-dev] case-insensitive #include warning

[Eric Niebler via cfe-dev]

On 4/20/16, 11:12 AM, "Reid Kleckner" <rnk at google.com> wrote:

> Thanks for the patch, this is definitely interesting! It looks pretty
> reasonable too. I'll check it out on phabricator when it's up.
>
> When you say the pre-processor is 2-3% slower, do you mean that the
> overall compile time of spirit etc got 2% longer, or just that the
> pre-processing step in a system like distcc got slower?

I mean that "clang -cc1 -Eonly all-spirit.cpp" got 2% slower, where all-spirit.cpp is a file that #include's every file in Boost.Spirit. That's just testing the preprocessor.


On 4/20/16, 11:27 AM, "Bruno Cardoso Lopes" <bruno.cardoso at gmail.com> wrote:

>> (Relaying second-hand information here). On OS X, I think realpath and
>> F_GETPATH actually return something from an xnu cache which may or may
>> not match the case of the underlying file depending on recent
>> operations.
>
> That's right!
>
>> Apparently for HFS filesystems, apparently fgetattrlist(ATTR_CMN_NAME)
>> will work, but even that's not guaranteed on all case-insensitive
>> filesystems (yet?).
>
> I'm not sure about "guaranteed on all case-insensitive", but this
> looks like the way to go on Darwin. Eric, could you take this into
> consideration on your patches?

Sure. I'm working on a new patch now.


\e