Skip to content

llvm/test/MC/AArch64/seh-* tests fail when VK_COFF_IMGREL32 changes value #144832

@MaskRay

Description

@MaskRay
Member

The enumerator VK_WEAKREF is no longer used. However, when I try removing it,

--- i/llvm/include/llvm/MC/MCExpr.h
+++ w/llvm/include/llvm/MC/MCExpr.h
@@ -199,7 +199,6 @@ public:
     VK_None,

     VK_SECREL,
-    VK_WEAKREF, // The link between the symbols in .weakref foo, bar

     VK_COFF_IMGREL32, // symbol@imgrel (image-relative)

I get failures related to AArch64 Win64EH. Perhaps something is sensitive to the value of VK_COFF_IMGREL32?

Failed Tests (4):
  LLVM :: MC/AArch64/seh-large-func-multi-epilog.s
  LLVM :: MC/AArch64/seh-large-func.s
  LLVM :: MC/AArch64/seh-packed-epilog.s
  LLVM :: MC/AArch64/seh-packed-unwind.s

@mstorsjo

Activity

mstorsjo

mstorsjo commented on Jul 5, 2025

@mstorsjo
Member

Sorry, I haven't had time to look at this yet. CC @efriedma-quic who also is quite familiar with this code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @mstorsjo@MaskRay@EugeneZelenko

        Issue actions

          llvm/test/MC/AArch64/seh-* tests fail when VK_COFF_IMGREL32 changes value · Issue #144832 · llvm/llvm-project