-
Notifications
You must be signed in to change notification settings - Fork 78
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
update llir/llvm to support 12.0 #169
Comments
This comment has been minimized.
This comment has been minimized.
LLVM 12.0 has now been released https://github.com/llvm/llvm-project/releases/tag/llvmorg-12.0.0 Release announcement: https://releases.llvm.org/12.0.0/docs/ReleaseNotes.html |
Compare ASM parser changes: $ wget https://github.com/llvm/llvm-project/archive/llvmorg-11.0.0.tar.gz
$ wget https://github.com/llvm/llvm-project/archive/llvmorg-12.0.0.tar.gz
$ tar zxf llvmorg-11.0.0.tar.gz
$ tar zxf llvmorg-12.0.0.tar.gz
$ git diff llvm-project-llvmorg-11.0.0/llvm/lib/AsmParser llvm-project-llvmorg-12.0.0/llvm/lib/AsmParser |
Instead of pasting the result, I would create a diff file, delete the updated part from it in PR. |
Post as a GitHub gist file? https://gist.github.com/ |
I want to track with PR, using diff file can see changes, maybe put it at updateand gist is personal I remember? Then we cannot track changes. |
You can keep it in
Fair point. Keep the diff in the llvm-12 branch is a good idea then. |
@mewmew would you like to start working on test cases updating? |
If you would like to try updating the test cases this time, then feel free. Otherwise, I can update them later on. I'll be heading out on vacation soon; will be tenting this weekend and then going for a longer trek in the south of Sweden starting next week. |
I had some time left over :) So sent a PR for updating the test cases: llir/testdata#9 There are currently 1 known parse error (see #195 (comment))
|
Translating AST to IR
l-tm testdata/llvm/test/DebugInfo/ARM/selectiondag-deadcode.ll
# Output:
# === [ testdata/llvm/test/DebugInfo/ARM/selectiondag-deadcode.ll ] =======================
#
# panic: support for parameter attribute *ast.StructRetAttr not yet implemented
#
# goroutine 1 [running]:
# github.com/llir/llvm/asm.(*generator).irParamAttribute(0x686bd8, {0x7f0ffd834390, 0xc00000e1a0})
# /home/u/life/projects/github.com/llir/llvm/asm/helper.go:630 +0x693 Edit: fixed in commit a84775f. |
NOTE: send pull requests to branch
llvm12
.Changes to LLVM IR
byref
attribute to better represent argument passing for the amdgpu_kernel calling convention.sret
attribute to continue work on removing pointer element types.The text was updated successfully, but these errors were encountered: