[PATCH] D150922: [KnownBits] Return `0` for poison {s,u}div inputs

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 18 17:40:44 PDT 2023


goldstein.w.n added inline comments.


================
Comment at: llvm/test/CodeGen/WebAssembly/pr59626.ll:18
+; CHECK-32-NEXT:    i32.store8 2
+; CHECK-32-NEXT:    local.get 1
 ; CHECK-32-NEXT:    local.get 0
----------------
I think this is a meaningless regression. The IR being tests evaluates to just  `ret i8 poison` @luke is this okay?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150922



More information about the llvm-commits mailing list