[PATCH] D106058: [DAG] Fold select(cond,binop(x,y),binop(x,z)) -> binop(x,select(cond,y,z))

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 15 06:15:46 PDT 2021


lebedev.ri accepted this revision.
lebedev.ri added a reviewer: tra.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Looks good to me.
I think NVPTX tests should just be adjusted beforehand.



================
Comment at: llvm/test/CodeGen/NVPTX/fast-math.ll:147
 
 ; CHECK-LABEL: repeated_div_recip_allowed
 define float @repeated_div_recip_allowed(i1 %pred, float %a, float %b, float %divisor) {
----------------
I think in all of these you want to have different divisors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106058



More information about the llvm-commits mailing list