[PATCH] D156691: [PEI][WebAssembly] Switch to backwards frame index elimination

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 15:47:39 PDT 2023


arsenm added a comment.

In D156691#4555665 <https://reviews.llvm.org/D156691#4555665>, @tlively wrote:

> Would it be possible to add a test that shows benefit from this? All the test changes in this diff seem to merely renumber virtual registers.

Probably not, any improvement implies there's a bug somewhere from not setting a kill flag. This avoids that maintenance cost


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156691



More information about the llvm-commits mailing list