This patch renames checkListEntryOperands() to checkOperandCount(), so
that we are able to check DWARF expression operands using the same
function.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
200 ms | linux > Polly.ScopInfo::Unknown Unit Message ("") |
Event Timeline
Comment Actions
Seems reasonable, but I think a better spelling of the name would be checkOperandNumber (no s) -- "check number of operands" vs "check operand number"
Comment Actions
I'd actually go one step further and call it checkOperandCount. I think checkOperandNumber to me suggests "check the number of the operand X". LGTM, with that tweak.