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

Auto-repair attributes of RV intrinsics #35

Open
simoll opened this issue May 15, 2019 · 0 comments
Open

Auto-repair attributes of RV intrinsics #35

simoll opened this issue May 15, 2019 · 0 comments

Comments

@simoll
Copy link
Member

simoll commented May 15, 2019

This is a reminder to...

a) Implement a pass that sets the right LLVM function attributes for RV builtins (RVIntrinsics)

  • rv_load should be readonly (SIMD codegen crashes for at least one Rodent traversal variant (not all variants are enabled by default) because the DA incorrectly believes the rv_load result shape is varying).
  • It might help if rv_any is annotated convergent (this is a potential workaround to stop LLVM from "hoisting ifs" involving rv_any, which may render the BOSCC/cif gadget ineffective before it hits RV).

b) Add this pass at the earliest insertion point when used with Clang.

c) Make sure RV itself only generates correct/complete RV intrinsic declarations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant