-
Notifications
You must be signed in to change notification settings - Fork 263
Open
Description
I noticed this warning with gcc 12.2.0:
ICECC[13224] 2024-03-18 14:34:38: argument -pedantic, forcing local preprocessing (try using -std=cXX instead of -std=gnuXX)
Indeed we have -Wpedantic BUT we also have -std=c++17.
The check is implemented here:
Lines 891 to 897 in 9d397ef
| // -pedantic doesn't work with remote preprocessing, if extensions to a named standard | |
| // are allowed. GCC allows GNU extensions by default, so let's check if a standard | |
| // other than eg gnu11 or gnu++14 was specified. | |
| if( seen_pedantic && !compiler_is_clang(job) && (!standard || str_startswith("gnu", standard)) ) { | |
| log_warning() << "argument -pedantic, forcing local preprocessing (try using -std=cXX instead of -std=gnuXX)" << endl; | |
| job.setBlockRewriteIncludes(true); | |
| } |
The standard variable is declared on top but never assigned:
Line 319 in 9d397ef
| const char *standard = nullptr; |
Is remote preprocessing supposed to work as described in the warning message/comment? If yes and it was overlooked, I can send a PR.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels