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

Handle LLVM intrinsic llvm.is_fpclass #1307

Open
alan-baker opened this issue Feb 16, 2024 · 0 comments
Open

Handle LLVM intrinsic llvm.is_fpclass #1307

alan-baker opened this issue Feb 16, 2024 · 0 comments

Comments

@alan-baker
Copy link
Collaborator

LLVM has started inferring llvm.is_fpclass (see https://llvm.org/docs/LangRef.html#llvm-is-fpclass) in some cases including the following CTS tests:

  • half/vstore_half_rtp
  • half/vstorea_half_rtp

In those cases it only used 0x40 (positive zero). Those tests passed with either zero though (e.g. using fcmp oeq 0). The remaining functionality should be implemented and tested.

alan-baker added a commit to alan-baker/clspv that referenced this issue Feb 16, 2024
* Handle switch to new debug info format
  * linked libraries are converted
  * newly created functions (with bodies) are converted
  * updated test expectations
* Bare minimum handling of llvm.is_fpclass (see google#1307) to unblock known
  cts failures
rjodinchr pushed a commit that referenced this issue Feb 16, 2024
* Update LLVM

* Handle switch to new debug info format
  * linked libraries are converted
  * newly created functions (with bodies) are converted
  * updated test expectations
* Bare minimum handling of llvm.is_fpclass (see #1307) to unblock known
  cts failures

* fix typo
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

No branches or pull requests

1 participant