[PATCH] D158018: [FunctionPropertiesAnalysis] Add operand type counts
Aiden Grossman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 15 15:24:30 PDT 2023
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
More information about the llvm-commits
mailing list