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

Can ikos support comparison with vector? #170

Open
neon12345 opened this issue Feb 15, 2021 · 1 comment
Open

Can ikos support comparison with vector? #170

neon12345 opened this issue Feb 15, 2021 · 1 comment
Labels
A-llvm-to-ar Area: Translation from LLVM to AR C-feature-request Category: Feature Request

Comments

@neon12345
Copy link

There is support for VectorType with translate_binary_operator for llvm::BinaryOperator but not for llvm::CmpInst.

Is it possible to transfer the vector handling from translate_binary_operator to translate_cmp and extend Type::is_unsigned_integer(), Type::is_signed_integer() for VectorType?

It seems to run this way but is it correct?

@arthaud
Copy link
Member

arthaud commented Feb 15, 2021

Hi @neon12345,

I'm not working on ikos anymore, but just to answer your question: yes, it should be possible to support llvm::VectorType in llvm::CmpInst. Note that the analysis ignores all VectorType instructions anyway, while being sound (For instance, https://github.com/NASA-SW-VnV/ikos/blob/master/analyzer/include/ikos/analyzer/analysis/execution_engine/numerical.hpp#L1337)

@arthaud arthaud added A-llvm-to-ar Area: Translation from LLVM to AR C-feature-request Category: Feature Request labels Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-llvm-to-ar Area: Translation from LLVM to AR C-feature-request Category: Feature Request
Projects
None yet
Development

No branches or pull requests

2 participants