[llvm] [mlgo-utils] Hoist entry script out to the correct directory (PR #146981)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 3 20:18:50 PDT 2025
boomanaiden154 wrote:
> I didn't realize these could also be installed through pip. From the looks of it, there's a top level wrapper that I can see that essentially just imports the main file and then executes it. This is the model that lit has well, with the exception that the source tree doesn't contain that wrapper in llvm that pip has created.
Can you build the package locally, install it, and test that the binaries work as expected?
> This PR would allow us to converge on one implementation that bazel and buck could use. If this is breaking some other mechanism, we can always do other internal hacks to get it working. I figured I'd try to see if we can have harmony first between the build systems first before resorting to other solutions.
I don't think this should break anything on the bazel/blaze side. I'm fine with this patch going in if it doesn't break any functionality in the pip package.
https://github.com/llvm/llvm-project/pull/146981
More information about the llvm-commits
mailing list