UnsafeAccessor works on non-static methods

[vitek-karas]

    class InstanceAccessors
    {
        public InstanceAccessors() { }

        [UnsafeAccessor(UnsafeAccessorKind.Constructor)]
        public extern InstanceAccessors UnsafeNonStaticAccessorForConstructor();
    }

   void Test()
   {
       (new InstanceAccessors()).InstanceAccessors(); // Works
   }

Per the proposal #81741, the attribute should only be effective on static methods. It's also not clear what it means on instance methods.

@AaronRobinsonMSFT

[ghost]

Tagging subscribers to this area: @dotnet/area-system-runtime-compilerservices
See info in area-owners.md if you want to be subscribed.

Issue Details

---

    class InstanceAccessors
    {
        public InstanceAccessors() { }

        [UnsafeAccessor(UnsafeAccessorKind.Constructor)]
        public extern InstanceAccessors UnsafeNonStaticAccessorForConstructor();
    }

   void Test()
   {
       (new InstanceAccessors()).InstanceAccessors(); // Works
   }

Per the proposal #81741, the attribute should only be effective on static methods. It's also not clear what it means on instance methods.

@AaronRobinsonMSFT

Author:	vitek-karas
Assignees:	-
Labels:	area-System.Runtime.CompilerServices
Milestone:	-

[AaronRobinsonMSFT]

the attribute should only be effective on static methods

Yep. I will add a run-time check for CoreCLR and a compile time check in NativeAOT.

/cc @fanyang-mono @jkotas