[PATCH] D116935: [IRBuilder] Introduce folder using inst-simplify, use for Or fold.

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Thanks for looking into this.
This seems conceptually fine to me.



================
Comment at: llvm/include/llvm/Analysis/InstSimplifyFolder.h:52-54
+    if (auto *LC = dyn_cast_or_null<Constant>(LHS))
+      if (auto *RC = dyn_cast_or_null<Constant>(RHS))
+        return ConstantFolder.FoldOr(LHS, RHS);
----------------
I don't think you need to do this explicitly?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116935