[PATCH] D158018: [FunctionPropertiesAnalysis] Add operand type counts

[Aiden Grossman via Phabricator via llvm-commits]

aidengrossman added inline comments.


================
Comment at: llvm/lib/Analysis/FunctionPropertiesAnalysis.cpp:134
+        } else if (isa<Argument>(Operand)) {
+          ArgumentOperandCount += Direction;
         }
----------------
jdoerfert wrote:
> The above should probably be a macro + a few invocations, also below. Anyway.
Can you explicate a little bit more on how you envision the macro? A macro defining a list of features that macros can then be plugged into for doing things like printing? Generating the operand counts with a macro seems a little counterintuitive to me since I believe it would be hard to preserve the semantics of the `if/else if/else` chain.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158018