Skip to content

Add (e)z80-clang compilers.#1566

Merged
partouf merged 1 commit intocompiler-explorer:mainfrom
adriweb:add-z80-clang
Apr 10, 2025
Merged

Add (e)z80-clang compilers.#1566
partouf merged 1 commit intocompiler-explorer:mainfrom
adriweb:add-z80-clang

Conversation

@adriweb
Copy link
Copy Markdown
Contributor

@adriweb adriweb commented Apr 8, 2025

These clang + LLVM backends are not upstreamed, so it's a bit customized.

I've tested locally and the binaries get installed as expected.

See also compiler-explorer/compiler-explorer#7546

These clang + LLVM backends are not upstreamed, so it's a bit customized.
@partouf partouf merged commit b7aefaf into compiler-explorer:main Apr 10, 2025
@partouf
Copy link
Copy Markdown
Member

partouf commented Apr 11, 2025

These didn't install last night, will have to investigate what's going on

@adriweb
Copy link
Copy Markdown
Contributor Author

adriweb commented Apr 11, 2025

Ah, that's odd considering I tested it fine locally and even the frontend worked, and I don't believe I manually changed anything...

Anyway there's some double bin folder in the path, from the error log, oops?
sudo: z80-clang-15.0.7/bin/bin/clang++: command not found
Maybe it's just the check_exe that needs to have bin/ removed.

Note that clang++ is a symlink to clang but I doubt this changes anything.

@mattgodbolt
Copy link
Copy Markdown
Member

Right, the two that fail are:

Failed:
  compilers/c++/z80-clang 15.0.0
  compilers/c++/z80-clang 15.0.7

I'm taking a quick look now.

@mattgodbolt
Copy link
Copy Markdown
Member

yeah; looks like you're unpacking by telling it the dir is ../bin/ - the check_exe is relative to this directory, so a check_exe of bin/clang means it's doing bin/bin/clang. I'll poke about and either change the unpack dir or fix the check_exe

@mattgodbolt
Copy link
Copy Markdown
Member

Looks like the tarball only has clang and clang++ -- does it really need to be in a bin directory?

@mattgodbolt
Copy link
Copy Markdown
Member

bollo:~/d/c/infra (main|✔) $ curl -sL https://github.com/CE-Programming/llvm-project/releases/download/nightly-rebase-v15/z80-clang-v1500-ubuntu-22.04Release.tar.gz | tar ztvf -

-rwxr-xr-x adriweb/staff   187 2025-04-08 03:49 ._clang
tar: Ignoring unknown extended header keyword 'LIBARCHIVE.xattr.com.apple.quarantine'
-rwxr-xr-x adriweb/staff 88274976 2025-04-08 03:49 clang
lrwxr-xr-x adriweb/staff        0 2025-04-08 17:57 clang++ -> clang

@mattgodbolt
Copy link
Copy Markdown
Member

I'm going to assume so, if only to make it look a bit more like a "normal" installation.

mattgodbolt added a commit that referenced this pull request Apr 14, 2025
mattgodbolt pushed a commit to compiler-explorer/compiler-explorer that referenced this pull request Apr 14, 2025
These clang + LLVM backends are not upstreamed but [a community
effort](https://github.com/CE-Programming/llvm-project/), so it's a bit
customized.

This requires compiler-explorer/infra#1566

_Edit: tested locally, works fine._
mattgodbolt pushed a commit to compiler-explorer/compiler-explorer that referenced this pull request Apr 21, 2025
These clang + LLVM backends are not upstreamed but [a community
effort](https://github.com/CE-Programming/llvm-project/), so it's a bit
customized.

This requires compiler-explorer/infra#1566

_Edit: tested locally, works fine._
mattgodbolt pushed a commit to detjonmataj/compiler-explorer that referenced this pull request Apr 25, 2025
These clang + LLVM backends are not upstreamed but [a community
effort](https://github.com/CE-Programming/llvm-project/), so it's a bit
customized.

This requires compiler-explorer/infra#1566

_Edit: tested locally, works fine._
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants