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

Emit remaining int operations #1702

Open
15 of 32 tasks
hugrbot opened this issue Nov 25, 2024 · 1 comment
Open
15 of 32 tasks

Emit remaining int operations #1702

hugrbot opened this issue Nov 25, 2024 · 1 comment
Assignees
Labels
llvm Pertains to LLVM emission

Comments

@hugrbot
Copy link
Collaborator

hugrbot commented Nov 25, 2024

Originally raised by @croyzor:

Having done some int ops We're still missing emission for some int ops from the arithmetic extension:

Done in #1835

  • ine,
  • imax_u,
  • imax_s,
  • imin_u,
  • imin_s,
  • iabs,
  • iand,
  • ior,
  • ixor,
  • inot,
  • ishl,
  • ishr,

In progress (#1839)

  • ipow

These should be easy (supported in LLVM)

  • iwiden_u,
  • iwiden_s,
  • inarrow_u,
  • inarrow_s,
  • itobool,
  • ifrombool,
  • idivmod_u,
  • idivmod_s,
  • imod_u,

These i'm not so sure about

  • imod_checked_u,
  • imod_checked_s,
  • idiv_checked_u,
  • idiv_checked_s,
  • idivmod_checked_u,
  • idivmod_checked_s,
  • irotl,
  • irotr,

These seem very hard

  • itostring_u,
  • itostring_s,
@ss2165
Copy link
Member

ss2165 commented Dec 24, 2024

priorities:
ine, ixor, ipow

github-merge-queue bot pushed a commit that referenced this issue Jan 6, 2025
More work on #1702. Adds `ine`, `iabs`, `imax_{s,u}`, `imin_{s,u}`,
`ishl`, `ishr`, `ixor`, `ior`, `inot`, `iand`

---------

Co-authored-by: Seyon Sivarajah <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
llvm Pertains to LLVM emission
Projects
None yet
Development

No branches or pull requests

3 participants