[PATCH] D130680: [IntegerDivision][AMDGPU] Use CreateLogicalOr to block poison propagation.

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 09:22:27 PDT 2022


aqjune added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/IntegerDivision.cpp:150
 static Value *generateUnsignedDivisionCode(Value *Dividend, Value *Divisor,
                                            IRBuilder<> &Builder) {
   // The basic algorithm can be found in the compiler-rt project's
----------------
Should `Dividend` and `Divisor` here be frozen to address the comment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130680



More information about the llvm-commits mailing list