docs(cpp/faq): Add RAM disk usage instructions for IntelliSense cache (ipch) - #8859
Devank Gupta (Devankguptaa) wants to merge 3 commits into
Conversation
|
@microsoft-github-policy-service agree |
|
Alexandra Kemper (@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 (@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. |
|
can you now review my PR?? |
|
Sean McManus (@sean-mcmanus) I discussed this with Alexandra Kemper (@AlexandraKemperMS) and because this is a very specific use case, she suggested to include this in the wiki over here: https://github.com/microsoft/vscode-cpptools/wiki/Optimizing-your-configuration WDYT? |
|
Sean McManus (@sean-mcmanus) can you check?? |
|
The wiki seems good to me. |
|
Nick Trogh (@ntrogh) can you merge my PR now?? |
|
Devank Gupta (@Devankguptaa) The decision was to include this guidance in the wiki (https://github.com/microsoft/vscode-cpptools/wiki/Optimizing-your-configuration) and not in the C++ docs. I'll close this PR - can you create a request in the microsoft/vscode-cpptools repo? |
|
Hi Nick Trogh (@ntrogh) 👋 |
|
Sean McManus (@sean-mcmanus) Can you provide guidance on how to best proceed? Should Devank Gupta (@Devankguptaa) file an issue with the wiki updates? |
|
Alexandra Kemper (@AlexandraKemperMS) Nick Trogh (@ntrogh) I don't think users can create a PR to update the Wiki? Or I could be mistaken...it might be possible via |
|
Sean McManus (@sean-mcmanus) I suggest creating an issue and providing the details for updating the wiki in there. |
|
Sean McManus (@sean-mcmanus) can you raise the issue? |
|
Sean McManus (@sean-mcmanus) should I have to raise the PR again or it will be automatically merged? |
|
Sean McManus (@sean-mcmanus) should I have to raise the PR again or it will be automatically merged? |
|
Devank Gupta (@Devankguptaa) Merged where? To the wiki? |
|
Sean McManus (@sean-mcmanus) Yes to the wiki |
|
Devank Gupta (@Devankguptaa) We can add it to the wiki page after someone reviews it. |
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