Merge pull request #377 from coffeemakingtoaster/master

rcoreilly · web-flow · commit 466844a4c500 · 2025-07-29T23:21:36.000-07:00
Migrate clang call from deprecated '-Ofast' to '-O3 -ffast-math'
diff --git a/cmd_build.go b/cmd_build.go
@@ -231,7 +231,7 @@ func runBuild(mode bind.BuildMode, cfg *BuildCfg) error {
 		}
 
 		cflags := strings.Fields(strings.TrimSpace(pycfg.CFlags))
-		cflags = append(cflags, "-fPIC", "-Ofast")
+		cflags = append(cflags, "-fPIC", "-O3", "-ffast-math")
 		if include, exists := os.LookupEnv("GOPY_INCLUDE"); exists {
 			cflags = append(cflags, "-I"+filepath.ToSlash(include))
 		}

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,7 @@ func runBuild(mode bind.BuildMode, cfg *BuildCfg) error {`
`231`	`231`	`}`
`232`	`232`
`233`	`233`	`cflags := strings.Fields(strings.TrimSpace(pycfg.CFlags))`
`234`		`- cflags = append(cflags, "-fPIC", "-Ofast")`
	`234`	`+ cflags = append(cflags, "-fPIC", "-O3", "-ffast-math")`
`235`	`235`	`if include, exists := os.LookupEnv("GOPY_INCLUDE"); exists {`
`236`	`236`	`cflags = append(cflags, "-I"+filepath.ToSlash(include))`
`237`	`237`	`}`