#import and indoes

Devang Patel dpatel@apple.com
Thu Aug 8 15:12:00 GMT 2002


On Thursday, August 8, 2002, at 11:25  AM, Zack Weinberg wrote:

>
> On Thu, Aug 08, 2002 at 10:37:05AM -0700, Devang Patel wrote:
>>>
>>> It would not be that hard to do the inode/dev check after opening the
>>> file, we call fstat() anyway.  And that'd be a good idea.  (Have to
>>> figure
>>> out a way to index the include cache on two different keys, though.)
>>
>> We do have local inode/dev check.  Here is the code snippet...
> [...]
>
> Yes, I am interested in seeing this cleaned up for submission.  You do
> need to do some work first - for instance, those static variables must
> go.
>
> In some workloads the existing cpp spends ~10% of runtime in splay
> tree lookups.  If we're dinking with this anyway, we might want to
> think about a better data structure to use.

I will cleanup as you suggested and prepare patch.

Before I start looking around, do you have any suggestion about
some other existing data structure which I can use to replace splay 
tree ?

Thank you,
-Devang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: text/enriched
Size: 1068 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20020808/78bb7dc8/attachment.bin>


More information about the Gcc-patches mailing list