[PATCH] D150142: [InstCombine] Add simplifications for div/rem with `i1` operands; PR62607

[Nikita Popov via Phabricator via llvm-commits]

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

LGTM



================
Comment at: llvm/test/Transforms/InstCombine/pr62607.ll:17
+; Function Attrs: nocallback nofree nosync nounwind willreturn memory(inaccessiblemem: readwrite)
+declare void @llvm.assume(i1 noundef) #1
+
----------------
Most of these declarations are dead?


================
Comment at: llvm/test/Transforms/InstCombine/pr62607.ll:19
+
+define i1 @function_0() {
+; CHECK-LABEL: @function_0(
----------------
Merge this into the other file with `@pr62607` name.


================
Comment at: llvm/test/Transforms/InstCombine/pr62607.ll:50
+  nocallback nofree nosync nounwind willreturn memory(inaccessiblemem:readwrite)
+}
----------------
Attributes shouldn't be needed either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150142