-
Notifications
You must be signed in to change notification settings - Fork 126
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
Action Space for LLVM 12 #788
Comments
Hi @cgao3, to adapt CompilerGym to use LLVM v12 rather than v10 you will need to modify the Action*.h files in https://github.com/facebookresearch/CompilerGym/tree/64bdd6cd39967d3d2fe5e6c72deb15e830b838bb/compiler_gym/envs/llvm/service/passes/10.0.0 . Those files were generated by a script that would assemble the pass list from the LLVM sources, rather than parsing the output of clang -O3 or similar. The script to generate the pass list is in this directory: You would need to run this on the LLVM 12 source tree to generate an updated set of Action*.h headers, then build from source using CMake configured to build against LLVM 12 rather than 10. Cheers, |
This is very helpful. Thanks so much. |
@ChrisCummins some follow up.. So I was trying to build from source with CMake. At last step, I failed with following error
Any idea how to solve it? I tried multiple different versions for |
@cgao3 I found same error, did you slover it?
|
I was able to bypass this problem. I used clang10, and was building gRPC from source externally. |
Some follow up.. I managed to build from source & made it support LLVM 12. After installing the new It gave me following error..
what could be reason for this? any ideas? @ChrisCummins |
Hi @cgao3, I'm sorry I missed your earlier messages, just catching up now. Congrats on your progress with the LLVM 12 port! That's great. The error you're reporting appears to be thrown here: You can add Cheers, |
This seems to be a GRPC problem. Are you sure you installed GRPC correctly? |
@sparks-code It seems to me gRPC is installed properly. But, do you know how to test gRPC separately? @ChrisCummins Here is a logging info after using
|
@cgao3, okay, looks like its expecting CompilerGym/compiler_gym/service/client_service_compiler_env.py Lines 721 to 724 in 9e0c0be
You may want to focus on BenchmarkCache.cc to see what it has populated at the time the error is raised to understand why the requested benchmark is "missing" Cheers, |
This comment was marked as resolved.
This comment was marked as resolved.
I also ran into an error with an external project.
I search the BUILD file of programl, it depends on llvm//10.0.0. May I ask if you have encountered such a problem, how to solve it? |
❓ What would be action space for LLVM 12
I am wondering what would be a suitable
action space
definition forllvm 12
. I know I can extract the "-O3" passes as following:However, it seems this set (94 passes) is incomplete. My question is then, what would be a complete version looks like? where can I find it?
Thanks
Attached. Action Space from LLVM 10 (current version)
Currently the
llvm-v0
env contains124
passes:The text was updated successfully, but these errors were encountered: