[PATCH] D114268: [InstrProf] Use i32 for GEP index from lowering llvm.instrprof.increment

Ellis Hoag via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 19 14:45:43 PST 2021


ellis added a comment.

In D114268#3143976 <https://reviews.llvm.org/D114268#3143976>, @MaskRay wrote:

>> Add names to pgo registers for clarity.
>
> This may increase memory usage, especially for large LTO applications. Unless this is very helpful I might omit it.

I'll omit it. Thanks for the quick review!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114268



More information about the cfe-commits mailing list