Skip to content

Commit

Permalink
Only build with gcc for now
Browse files Browse the repository at this point in the history
Newer clang has issues with valgrind wrt DWARF5 support, and two builds
may hit a race condition with the repository update anyway.
  • Loading branch information
zeux committed Jul 1, 2022
1 parent 6efe416 commit 071883d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest]
compiler: [g++, clang++]
compiler: [g++]
benchResultsRepo:
- { name: "luau-lang/benchmark-data", branch: "main" }

Expand Down
3 changes: 1 addition & 2 deletions bench/bench.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,7 @@ def getVmOutput(cmd):
file = open(os.path.join(scriptdir, "callgrind.out"), "r")
lines = file.readlines()
return getCallgrindOutput(lines)
except e:
print(e)
except:
return ""
else:
with subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True, cwd=scriptdir) as p:
Expand Down

0 comments on commit 071883d

Please sign in to comment.