[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