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

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 02:10:14 PDT 2023


foad added a comment.

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

> LGTM, then. Is there somewhere I can read more about the context for this change?

Thanks. I am not aware of a good overall summary, but there are various comments scattered around like:

  include/llvm/CodeGen/MachinePassRegistry.def
  71-// LiveVariables currently requires pure SSA form.
  72:// FIXME: Once TwoAddressInstruction pass no longer uses kill flags,
  73-// LiveVariables can be removed completely, and LiveIntervals can be directly
  74-// computed. (We still either need to regenerate kill flags after regalloc, or
  75-// preferably fix the scavenger to not depend on them).


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