Skip to content

Add vm.bitcast ops for i32<->f32 and i64<->f64.#6664

Merged
ScottTodd merged 6 commits intoiree-org:mainfrom
ScottTodd:vm-bitcast
Aug 10, 2021
Merged

Add vm.bitcast ops for i32<->f32 and i64<->f64.#6664
ScottTodd merged 6 commits intoiree-org:mainfrom
ScottTodd:vm-bitcast

Conversation

@ScottTodd
Copy link
Copy Markdown
Member

@ScottTodd ScottTodd commented Aug 5, 2021

Working towards #6629

Also included a conversion for standard bitcast -> vm.bitcast.* ops.

See also https://reviews.llvm.org/D105376 for std.bitcast, which has more details about endianness and includes some fold/canonicalization patterns. We have a more restrictive set of types to support here.

@ScottTodd ScottTodd added compiler/dialects Relating to the IREE compiler dialects (flow, hal, vm) runtime/vm IREE bytecode VM interpreter labels Aug 5, 2021
@ScottTodd ScottTodd requested a review from benvanik August 5, 2021 22:11
@google-cla google-cla Bot added the cla: yes label Aug 5, 2021
ScottTodd added a commit that referenced this pull request Aug 10, 2021
Working towards #6629

Depends on https://reviews.llvm.org/D105376

This requires no extra changes (beyond the dependent change in LLVM) to support splatting floating point _constants_. It will need a lowering from `std.bitcast` to `vm.bitcast` (#6664) for variable values.
Comment thread iree/vm/ops.h Outdated
@ScottTodd ScottTodd enabled auto-merge (squash) August 10, 2021 15:01
@ScottTodd
Copy link
Copy Markdown
Member Author

Presubmit passed but GitHub service degradation swallowed the completion signal. Merging manually.

@ScottTodd ScottTodd disabled auto-merge August 10, 2021 17:58
@ScottTodd ScottTodd merged commit 62f13b2 into iree-org:main Aug 10, 2021
@ScottTodd ScottTodd deleted the vm-bitcast branch August 10, 2021 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compiler/dialects Relating to the IREE compiler dialects (flow, hal, vm) runtime/vm IREE bytecode VM interpreter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants