-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LLVM 13, Ubuntu 22.04 #103
Conversation
7442b01
to
c1b9a34
Compare
f I run build the new Docker container as
I see
Not sure what's wrong yet. |
c1b9a34
to
506074f
Compare
Looks like LLVM 13 switched to the new pass manager, have to explicitly disable that on the command line. Done in |
6fde100
to
6693c03
Compare
Ubuntu 20.04 doesn't appear to have apt packages for LLVM 13, so might as well upgrade now.
6693c03
to
cfabefa
Compare
Oof, so many golden test file changes. To see which ones might be relevant, I'm using this script: from subprocess import check_output
out = check_output(["git", "diff", "--numstat", "HEAD", "HEAD~1"])
files = set()
for line in out.decode("utf-8").splitlines():
if "gold" not in line:
continue
(add, rm, nm) = line.split()
if abs(int(add) - int(rm)) > 1:
files.add(nm)
for f in files:
print(f) which gives me
Which is still a ton... |
...switches-O1-O1-fno-discard-value-names.1-callsite.subset_lift.alloc_may_alias_ctx.golden.csv
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first two files I looked at were more improvements in LLVM optimizations. I can't feasibly look through all of these, so I'm going to go ahead and merge.
Towards #12