compileopts: remove workaround for LLVM 16

aykevl · aykevl · commit 7c9025c3a8d1 · 2024-02-19T15:32:31.000+01:00
This workaround is no longer needed now that we've switched to LLVM 17 where this bug has been fixed upstream: espressif/llvm-project#84
diff --git a/compileopts/config.go b/compileopts/config.go
@@ -286,10 +286,6 @@ func (c *Config) CFlags(libclang bool) []string {
 		cflags = append(cflags,
 			"-resource-dir="+resourceDir,
 		)
-		if strings.HasPrefix(c.Triple(), "xtensa") {
-			// workaround needed in LLVM 16, see: https://github.com/espressif/llvm-project/issues/83
-			cflags = append(cflags, "-isystem", filepath.Join(resourceDir, "include"))
-		}
 	}
 	switch c.Target.Libc {
 	case "darwin-libSystem":

Original file line number	Diff line number	Diff line change
`@@ -286,10 +286,6 @@ func (c *Config) CFlags(libclang bool) []string {`
`286`	`286`	`cflags = append(cflags,`
`287`	`287`	`"-resource-dir="+resourceDir,`
`288`	`288`	`)`
`289`		`- if strings.HasPrefix(c.Triple(), "xtensa") {`
`290`		`- // workaround needed in LLVM 16, see: https://github.com/espressif/llvm-project/issues/83`
`291`		`- cflags = append(cflags, "-isystem", filepath.Join(resourceDir, "include"))`
`292`		`- }`
`293`	`289`	`}`
`294`	`290`	`switch c.Target.Libc {`
`295`	`291`	`case "darwin-libSystem":`