Skip to content
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

more missing -mcpu values for powerpc #1820

Open
nickdesaulniers opened this issue Mar 20, 2023 · 2 comments
Open

more missing -mcpu values for powerpc #1820

nickdesaulniers opened this issue Mar 20, 2023 · 2 comments
Labels
[ARCH] powerpc This bug impacts ARCH=powerpc [BUG] llvm A bug that should be fixed in upstream LLVM

Comments

@nickdesaulniers
Copy link
Member

See also 4b10306.

So cell, e6500, 405, 464, 476, 860, e300c2, and e300c3 support is missing from clang.

These configs set the -mcpu value in

config TARGET_CPU
string
depends on TARGET_CPU_BOOL
default "cell" if CELL_CPU
default "power5" if POWER5_CPU
default "power6" if POWER6_CPU
default "power7" if POWER7_CPU
default "power8" if POWER8_CPU
default "power9" if POWER9_CPU
default "860" if 860_CPU
default "e300c2" if E300C2_CPU
default "e300c3" if E300C3_CPU
default "G4" if G4_CPU
.

cc @nemanjai

@nickdesaulniers nickdesaulniers added [BUG] llvm A bug that should be fixed in upstream LLVM [ARCH] powerpc This bug impacts ARCH=powerpc labels Mar 20, 2023
@nemanjai
Copy link

Since we don't have code generation for these, I would rather not accept them in Clang/LLVM. Are these configurations actually used by anyone or are these just remnants of something that hasn't been used in years? If the latter, can they just be removed?

@nathanchance
Copy link
Member

Are these configurations actually used by anyone or are these just remnants of something that hasn't been used in years?

The configurations are still in the tree, so people build them. Whether or not the resulting kernels are actually being booted and used in the wild, let alone with clang, is a completely different story and it is probably hard to figure out, since many hobbyists are not vocal about their usage as long as things stay working. The 32-bit processors are a dying breed obviously (although seemingly still sold in certain applications), so it is probably a waste of resources to try and implement code generation support for them.

For the record, this is being workaround in the kernel by preventing those configurations from being selected:

https://git.kernel.org/linus/4b10306e98456aed03cad75ce467e8b1efdccca0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[ARCH] powerpc This bug impacts ARCH=powerpc [BUG] llvm A bug that should be fixed in upstream LLVM
Projects
None yet
Development

No branches or pull requests

3 participants