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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 06:26:49 PDT 2017


spatel added a comment.

Is there an advantage to this vs. just putting the flags in the base SDNode class? We're going to eventually want to use the flags for more than unary and binary nodes (eg, FMA), so I'd prefer to just go directly to that step.


Repository:
  rL LLVM

https://reviews.llvm.org/D32527





More information about the llvm-commits mailing list