-
Notifications
You must be signed in to change notification settings - Fork 5.3k
docs(cpp/faq): Add RAM disk usage instructions for IntelliSense cache (ipch) #8859
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@microsoft-github-policy-service agree |
@AlexandraKemperMS Can you review this PR? |
The issue microsoft/vscode-cpptools#13890 also mentions the browse database, e.g. C_Cpp.default.browse.databaseFilename. |
This doesn't seem like a good use of memory to me. I think it would be a better to use the memory for the IntelliSense processes themselves (which memory map the ipch files, so by having the ipch files in memory you're essentially using twice as much memory). Users who are concerned with ipch writing should set C_Cpp.intelliSenseCacheSize to 0. If the cache is using memory, then it'll cause few IntelliSense processes to be available in memory. Do you have any data or performance data for scenarios where this actually improves performance? ...well, it would improve performance if you had a set of files that were frequently opened/closed causing the IntelliSense process to shutdown/restart frequently...but the user might be better off just keeping the files open (keep the cpptools-srv process running). |
The comment |
a7effd4
to
356d3fd
Compare
Thanks for the feedback, @sean-mcmanus . I’ve updated the FAQ:
|
can you review my PR?? |
I think our team might be busy right now. I might be able to review it later. |
This PR updates the C/C++ FAQ to clarify how developers can place the IntelliSense cache (ipch) on a RAM disk.
Changes made
Why this is useful
Checklist