[PATCH] D32527: Generalize flag carrying SDNodes beyond binary ops. NFC.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:5964
   case 0: return getNode(Opcode, DL, VT);
-  case 1: return getNode(Opcode, DL, VT, Ops[0]);
+  case 1: return getNode(Opcode, DL, VT, Ops[0], Flags);
   case 2: return getNode(Opcode, DL, VT, Ops[0], Ops[1], Flags);
----------------
spatel wrote:
> Should we wait to make this change until there is a transform that uses it (and therefore a test to prove it)?
Makes sense, I'll move this change to the next patch in the series. 


Repository:
  rL LLVM

https://reviews.llvm.org/D32527