[PATCH] D149423: [ValueTracking] Use knownbits interface for determining if `div`/`rem` are safe to speculate

Alexander Kornienko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 15:29:18 PDT 2023


alexfh added a comment.

  $ clang-6cdc229a64be8dacba40d30a9032c14f51ee30c0 -fsanitize=memory -O2 q.cc -S -emit-llvm -o good.ll
  $ clang-6c667abf3294d61e4fbe1238e1755c79f7547f1b -fsanitize=memory -O2 q.cc -S -emit-llvm -o bad.ll
  $ diff -u good.ll bad.ll
  --- good.ll
  +++ bad.ll
  @@ -11,8 +11,10 @@
     %call = tail call noundef i32 @_Z1qv()
     %spec.store.select = tail call i32 @llvm.umax.i32(i32 %call, i32 1)
     %1 = icmp ugt i32 %spec.store.select, 8192
  -  %div = udiv i32 8192, %spec.store.select
  -  %spec.store.select4 = select i1 %1, i32 1, i32 %div
  +  %div.rhs.trunc = trunc i32 %spec.store.select to i16
  +  %div7 = udiv i16 8192, %div.rhs.trunc
  +  %div.zext = zext i16 %div7 to i32
  +  %spec.store.select4 = select i1 %1, i32 1, i32 %div.zext
     ret i32 %spec.store.select4
   }
   
  @@ -40,4 +42,4 @@
   
   !0 = !{i32 1, !"wchar_size", i32 4}
   !1 = !{i32 7, !"uwtable", i32 2}
  -!2 = !{!"clang version trunk (6cdc229a64be8dacba40d30a9032c14f51ee30c0)"}
  +!2 = !{!"clang version trunk (6c667abf3294d61e4fbe1238e1755c79f7547f1b)"}


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149423



More information about the llvm-commits mailing list