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

LLVM mangling error with powi #4480

Closed
chalcolith opened this issue Jan 26, 2024 · 4 comments · Fixed by #4481
Closed

LLVM mangling error with powi #4480

chalcolith opened this issue Jan 26, 2024 · 4 comments · Fixed by #4481
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@chalcolith
Copy link
Member

The following program:

actor Main
  new create(env: Env) =>
    let a: F64 = 3.14
    let b: F64 = a.powi(-2)
    env.out.print("B is now " + b.string())

Gives LLVM errors when compiling on Linux:

0.58.0-a161b7c [release]
Compiled with: LLVM 15.0.7 -- Clang-16.0.6-x86_64
Defaults: pic=true
Intrinsic name not mangled correctly for type arguments! Should be: llvm.powi.f64.i32
ptr @llvm.powi.f64
Intrinsic name not mangled correctly for type arguments! Should be: llvm.powi.f64.i32
ptr @llvm.powi.f64
Error:
Module verification failed: Intrinsic name not mangled correctly for type arguments! Should be: llvm.powi.f64.i32
ptr @llvm.powi.f64
Intrinsic name not mangled correctly for type arguments! Should be: llvm.powi.f64.i32
ptr @llvm.powi.f64

    Info:
Please file an issue ticket. Use --noverify to bypass this error.
@chalcolith chalcolith added bug Something isn't working needs investigation This needs to be looked into before its "ready for work" labels Jan 26, 2024
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Jan 26, 2024
@SeanTAllen SeanTAllen added help wanted Extra attention is needed good first issue Good for newcomers labels Jan 26, 2024
@SeanTAllen
Copy link
Member

This is a very simple fix, I'll open a fix for it after I verify that it fails with a test, but there might be others we should review for.

@chalcolith
Copy link
Member Author

I already have a fix.

@SeanTAllen
Copy link
Member

@chalcolith can you get the f32 powi as well?

@SeanTAllen SeanTAllen removed help wanted Extra attention is needed needs investigation This needs to be looked into before its "ready for work" labels Jan 26, 2024
@chalcolith
Copy link
Member Author

Yes

@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants