[PATCH] D34986: [PowerPC] avoid redundant analysis while lowering an immediate; NFC
Hiroshi Inoue via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 12 05:05:50 PDT 2017
inouehrs added a comment.
@echristo In addition to `getInt64Count`, I feel that `getInt64` is also somewhat confusing since we have `getI64Imm` for other purpose.
How about `selectI64Imm` and `selectI64ImmInstrCount` instead of `getInt64` and `getInt64Count`?
Alternative names are `getInt64MaterializationInstr` and `getInt64MaterializationInstrCount` (suggested by @nemanjai)
Do you have any suggestion?
Repository:
rL LLVM
https://reviews.llvm.org/D34986
More information about the llvm-commits
mailing list