iree-org / iree-llvm-sandbox

A sandbox for quick iteration and experimentation on projects related to IREE, MLIR, and LLVM
Apache License 2.0
54 stars 31 forks source link

[Substrait] Decide how to check for number of ops in module. #815

Open ingomueller-net opened 7 months ago

ingomueller-net commented 7 months ago

This is related to the following snippet:

if (llvm::range_size(body.getOps()) != 1)

Original comment:

          This is a little wasteful/one could early exit without computing total size.

_Originally posted by @jpienaar in https://github.com/iree-org/iree-llvm-sandbox/pull/797#discussion_r1511031182_