Skip to content

Commit

Permalink
Don't pass -fwrapv to clang on Windows (GH-17458)
Browse files Browse the repository at this point in the history
This is apparently not supported there; the VS supplied clang version
18.1.8, reports:

`clang-cl: warning: unknown argument ignored in clang-cl: '-fwrapv' [-Wunknown-argument]`
  • Loading branch information
cmb69 authored Jan 13, 2025
1 parent 788128a commit aac7190
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions ext/date/config.w32
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,6 @@ PHP_DATE = "yes";
ADD_SOURCES("ext/date/lib", "astro.c timelib.c dow.c parse_date.c parse_posix.c parse_tz.c tm2unixtime.c unixtime2tm.c parse_iso_intervals.c interval.c", "date");

ADD_FLAG('CFLAGS_DATE', "/wd4244");
if (CLANG_TOOLSET) {
ADD_FLAG('CFLAGS_BD_EXT_DATE_LIB', "-fwrapv");
}

var tl_config = FSO.CreateTextFile("ext/date/lib/timelib_config.h", true);
tl_config.WriteLine("#include \"config.w32.h\"");
Expand Down

0 comments on commit aac7190

Please sign in to comment.