[PATCH] D74378: [MLIR] Add std.assume_align op.

[River Riddle via Phabricator via llvm-commits]

rriddle accepted this revision.
rriddle added a comment.
This revision is now accepted and ready to land.

Looks great thanks!



================
Comment at: mlir/lib/Dialect/StandardOps/Ops.cpp:2773
+  unsigned alignment = op.alignment().getZExtValue();
+  if ((alignment & (alignment - 1)) != 0)
+    return op.emitOpError("alignment must be power of 2");
----------------
llvm::isPowerOf2_32

?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74378/new/

https://reviews.llvm.org/D74378