#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