[PATCH] D105344: [DAGCombiner] Fold SETCC(FREEZE(x),const) to FREEZE(SETCC(x,const)) if SETCC is used by BRCOND

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 17 23:17:28 PDT 2021


aqjune added a comment.

This patch is fixed to deal with pulling out FREEZE from SETCC only. The removal of FREEZE is delegated to visitBRCOND's simplification.



================
Comment at: llvm/test/CodeGen/X86/setcc-freeze.ll:31
 ; CHECK-NEXT:    testb %al, %al
 ; CHECK-NEXT:    jne .LBB1_2
 ; CHECK-NEXT:  # %bb.1: # %A
----------------
These suboptimal codes were being generated even if `%cond` was not frozen as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105344



More information about the llvm-commits mailing list