[PATCH] D29014: [SelDag] Implement FREEZE node

Mon Oct 14 08:43:23 PDT 2019

lebedev.ri added subscribers: spatel, RKSimon, craig.topper.
lebedev.ri added a comment.

I believe vector legalization tests are missing.

Cost model handling might be missing (i would guess they should be treated as free)

This looks reasonable-ish, but i don't have much knowledge on this part, sadly.
I'm wondering if @RKSimon / @spatel / @craig.topper might want to help review this?

================
Comment at: include/llvm/CodeGen/ISDOpcodes.h:179

+    // FREEZE - FREEZE(VAL) returns an arbitrary integer if VAL is UNDEF (or
+    // is evaluated to UNDEF), or returns VAL otherwise. Note that each
----------------
s/integer/value/

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

https://reviews.llvm.org/D29014