[PATCH] D29014: [SelDag] Add FREEZE

Tue Mar 24 05:52:55 PDT 2020

aqjune marked an inline comment as done.
aqjune added inline comments.

================
Comment at: llvm/test/CodeGen/X86/freeze-legalize.ll:8-12
+; CHECK-NEXT:    movl $303174162, %eax ## imm = 0x12121212
+; CHECK-NEXT:    movl $875836468, %ecx ## imm = 0x34343434
+; CHECK-NEXT:    movl $1448498774, %edx ## imm = 0x56565656
+; CHECK-NEXT:    xorl %eax, %edx
+; CHECK-NEXT:    movl $2021161080, %eax ## imm = 0x78787878
----------------
aqjune wrote:
> spatel wrote:
> > Do we need to add basic simplify / constant folding for SDAG ?
> > freeze ( Constant ) --> Constant
> Yes, I agree it will be great. Do you want to make this patch contain the change as well?
Or I can land this first, and add the simplify / constant folding for SDAG.
I prefer incrementally making things because this patch itself is a big change.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D29014