Use attribute xml for security attribute stripping

[sbomer]

Context: https://github.com/dotnet/runtime/pull/96363/files#r1449983607

ILLink has a hard-coded list of security attributes to remove:

runtime/src/tools/illink/src/linker/Linker.Steps/RemoveSecurityStep.cs

Lines 67 to 75 in de1e529

	switch (attr_type.Name) {
	case "SecurityCriticalAttribute":
	case "SecuritySafeCriticalAttribute":
	case "SuppressUnmanagedCodeSecurityAttribute":
	case "DynamicSecurityMethodAttribute":
	case "UnverifiableCodeAttribute":
	case "AllowPartiallyTrustedCallersAttribute":
	case "SecurityTransparentAttribute":
	case "SecurityRulesAttribute":

We should consider using the same approach we use for other attributes (the ILLink.LinkAttributes.xml) for consistency, and to get the same behavior in NativeAot.

Tagging subscribers to this area: @agocke, @sbomer, @vitek-karas
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Context: https://github.com/dotnet/runtime/pull/96363/files#r1449983607

ILLink has a hard-coded list of security attributes to remove:

runtime/src/tools/illink/src/linker/Linker.Steps/RemoveSecurityStep.cs

Lines 67 to 75 in de1e529

	switch (attr_type.Name) {
	case "SecurityCriticalAttribute":
	case "SecuritySafeCriticalAttribute":
	case "SuppressUnmanagedCodeSecurityAttribute":
	case "DynamicSecurityMethodAttribute":
	case "UnverifiableCodeAttribute":
	case "AllowPartiallyTrustedCallersAttribute":
	case "SecurityTransparentAttribute":
	case "SecurityRulesAttribute":

We should consider using the same approach we use for other attributes (the ILLink.LinkAttributes.xml) for consistency, and to get the same behavior in NativeAot.

Author:	sbomer
Assignees:	-
Labels:	untriaged, area-Tools-ILLink, needs-area-label
Milestone:	-

[agocke]

Should we just drop support for this entirely? I didn't think we wanted to support attribute trimming at all...