[PATCH] D155588: [NFC][AMDGPULowerModuleLDSPass] Cleanup of getTableLookupKernelIndex

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 07:09:02 PDT 2023


JonChesterfield added a comment.

Destructuring bind on the emplace sounds good. I don't particularly mind the repeated calls to operator[] but the signal to noise ratio is about the same and this is quicker.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155588



More information about the llvm-commits mailing list