Skip to content

Commit

Permalink
fix: Remove TAKES_ARG for /Fp and /Yu MSVC options (ccache#1453)
Browse files Browse the repository at this point in the history
/Fp and /Yu don't allow space between option and option value.

Fixes ccache#1452
  • Loading branch information
silverqx authored May 17, 2024
1 parent 44abf78 commit 1bcde8b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/ccache/compopt.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ const CompOpt compopts[] = {
{"-F", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH},
{"-FI", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-FU", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-Fp", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-Fp", AFFECTS_CPP | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-G", TAKES_ARG},
{"-I", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH},
{"-L", TAKES_ARG},
Expand All @@ -97,8 +97,8 @@ const CompOpt compopts[] = {
{"-Xcompiler", AFFECTS_CPP | TAKES_ARG}, // nvcc
{"-Xlinker", TAKES_ARG | TAKES_CONCAT_ARG | AFFECTS_COMP},
{"-Xpreprocessor", AFFECTS_CPP | TAKES_ARG},
{"-Yc", AFFECTS_CPP | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-Yu", AFFECTS_CPP | TAKES_ARG | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-Yc", AFFECTS_CPP | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-Yu", AFFECTS_CPP | TAKES_CONCAT_ARG | TAKES_PATH}, // msvc
{"-all_load", AFFECTS_COMP},
{"-analyze", TOO_HARD}, // Clang
{"-arch", TAKES_ARG},
Expand Down

0 comments on commit 1bcde8b

Please sign in to comment.