Skip to content
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

tools: add msvc /P output to .gitignore #35735

Open
wants to merge 1 commit into
base: master
from

Conversation

@gengjiawen
Copy link
Member

@gengjiawen gengjiawen commented Oct 21, 2020

This is pretty useful when debug missing symbol on windows.

See https://docs.microsoft.com/en-us/cpp/build/reference/p-preprocess-to-a-file?view=vs-2019

@gengjiawen
Copy link
Member Author

@gengjiawen gengjiawen commented Oct 23, 2020

@targos
Copy link
Member

@targos targos commented Oct 23, 2020

How did you add the /P argument?

@gengjiawen
Copy link
Member Author

@gengjiawen gengjiawen commented Oct 23, 2020

How did you add the /P argument?

diff --git a/tools/v8_gypfiles/v8.gyp b/tools/v8_gypfiles/v8.gyp
index fd786d024e..696474c053 100644
--- a/tools/v8_gypfiles/v8.gyp
+++ b/tools/v8_gypfiles/v8.gyp
@@ -203,7 +203,7 @@
   'target_defaults': {
     'msvs_settings': {
       'VCCLCompilerTool': {
-        'AdditionalOptions': ['/utf-8']
+        'AdditionalOptions': ['/utf-8', '/P']
       }
     },
   },

Recently use case is #35433. The clang support maybe require us to patch v8 again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.