compileopts: remove workaround for LLVM 16

aykevl · deadprogram · commit a2588d8db313 · 2024-02-19T23:29:44.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
@@ -280,10 +280,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
`@@ -280,10 +280,6 @@ func (c *Config) CFlags(libclang bool) []string {`
`280`	`280`	`cflags = append(cflags,`
`281`	`281`	`"-resource-dir="+resourceDir,`
`282`	`282`	`)`
`283`		`- if strings.HasPrefix(c.Triple(), "xtensa") {`
`284`		`- // workaround needed in LLVM 16, see: https://github.com/espressif/llvm-project/issues/83`
`285`		`- cflags = append(cflags, "-isystem", filepath.Join(resourceDir, "include"))`
`286`		`- }`
`287`	`283`	`}`
`288`	`284`	`switch c.Target.Libc {`
`289`	`285`	`case "darwin-libSystem":`