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.

Sign up for GitHub

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

Jump to bottom

[LLVM] Missing libcalls #255

Closed
4 of 5 tasks
adriweb opened this issue Apr 18, 2020 · 5 comments
Closed
4 of 5 tasks

[LLVM] Missing libcalls #255

adriweb opened this issue Apr 18, 2020 · 5 comments
Labels
bug enhancement

Comments

@adriweb
Copy link
Member

adriweb commented Apr 18, 2020 •
edited by runer112
Loading

From various tests on the PB, here's what I see that's left to take care of before a release:

Seen in "real projects":

  • _ceilf
  • _fabsf
  • _floorf
  • _roundf
  • __llcmpzero (but probably other __ll stuff too)

Seen by manually testing from Z80ISelLowering.cpp:

  • __bbitrev
  • __bpopcnt
  • __dadd
  • __dcmp
  • __ddiv
  • __dmul
  • __dneg
  • __drem
  • __dsub
  • __dtof
  • __dtol
  • __dtoll
  • __dtoll
  • __dtoul
  • __frem
  • __ftod
  • __ibitrev
  • __ipopcnt
  • __lbitrev
  • __lladd
  • __lland
  • __llbitrev
  • __llcmpu
  • __lldivs
  • __lldivu
  • __llmulu
  • __llneg
  • __llnot
  • __llor
  • __llpopcnt
  • __llrems
  • __llremu
  • __llshl
  • __llshrs
  • __llshru
  • __llsub
  • __lltod
  • __llxor
  • __lpopcnt
  • __ltod
  • __sbitrev
  • __spopcnt
  • __ulltod
  • __ultod
@adriweb
Copy link
Member Author

adriweb commented Aug 18, 2020 •
edited
Loading

aa935b1 fixed a few: _ceilf, _fabsf, _roundf.
Then 4829561 and b3e4ef5 some other.

@mateoconlechuga
Copy link
Member

Closed in favor of #276

@runer112
Copy link
Member

runer112 commented Sep 11, 2020 •
edited
Loading

@mateoconlechuga There are still some in the list that aren't double, long double, or long long though?

@adriweb
Copy link
Member Author

adriweb commented Sep 11, 2020 •
edited by mateoconlechuga
Loading

Those would be:

  • __bbitrev
  • __sbitrev
  • __ibitrev
  • __lbitrev

And:

  • __bpopcnt
  • __spopcnt
  • __ipopcnt
  • __lpopcnt

And:

  • __frem

Right?

@adriweb
Copy link
Member Author

adriweb commented Dec 21, 2020

Closing this in favor of #276, although what's the deal with _frem?

@adriweb adriweb closed this as completed Dec 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug enhancement
Milestone
No milestone
Development

No branches or pull requests
3 participants
@adriweb @runer112 @mateoconlechuga