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

[Amara Emerson via Phabricator via llvm-commits]

aemerson added a comment.

In https://reviews.llvm.org/D32527#738013, @spatel wrote:

> 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.


I'm not the original author of the code but I think this is due to the extra 2 bytes of storage needed for the flags in each SDNode. With the current solution we only incur this cost if we have flags to store.


Repository:
  rL LLVM

https://reviews.llvm.org/D32527