[all-commits] [llvm/llvm-project] 726e46: [Statepoint] Treat undef operands less specially

Matt Arsenault via All-commits all-commits at lists.llvm.org
Mon Dec 16 21:55:17 PST 2024


  Branch: refs/heads/users/arsenm/stackmap-undef
  Home:   https://github.com/llvm/llvm-project
  Commit: 726e4688dda7a5aa4d96a4705005547fc5cd51ae
      https://github.com/llvm/llvm-project/commit/726e4688dda7a5aa4d96a4705005547fc5cd51ae
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-12-17 (Tue, 17 Dec 2024)

  Changed paths:
    M llvm/lib/CodeGen/FixupStatepointCallerSaved.cpp
    M llvm/lib/CodeGen/StackMaps.cpp
    A llvm/test/CodeGen/X86/stackmap-undef-operand-anyregcc.mir
    M llvm/test/CodeGen/X86/statepoint-fixup-undef.mir

  Log Message:
  -----------
  [Statepoint] Treat undef operands less specially

This reverts commit f7443905af1e06eaacda1e437fff8d54dc89c487.

This is to avoid an assertion if an undef operand appears in a
stackmap. This is important to avoid hitting verifier errors
when register allocation starts adding undefs in error scenarios.

Rather than trying to treat undef operands as special, leave them
alone and avoid producing an invalid spill. It would a bit more
precise to produce a spill of an undef register here, but that's not
exposed through the storeRegToStackSlot API.

https://reviews.llvm.org/D122605

This was an alternative to https://reviews.llvm.org/D122582



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list