Skip to content

Conversation

@viewizard
Copy link
Member

Could be used for compile on x64 host for arm64 target (for example, during Tizen OS image creation).

Related to #121175, another approach for disable optimistic instruction set to be added.

CC @gbalykov @tannergooding @MichalStrehovsky

Could be used for compile on x64 host for arm64 target (for example, during Tizen OS image creation).
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 30, 2025
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Oct 30, 2025
@am11
Copy link
Member

am11 commented Oct 30, 2025

It would be nice to also update the help text with native and optimistic options (ilc --help, crossgen2 --help):

Console.WriteLine("The allowable values for the --instruction-set option are described in the table below. Each architecture has a different set of valid " +
"instruction sets, and multiple instruction sets may be specified by separating the instructions sets by a ','. For example 'avx,aes,apx'");
foreach (string arch in ValidArchitectures)
{
TargetArchitecture targetArch = Helpers.GetTargetArchitecture(arch);
bool first = true;
foreach (var instructionSet in Internal.JitInterface.InstructionSetFlags.ArchitectureToValidInstructionSets(targetArch).DistinctBy((instructionSet) => instructionSet.Name, StringComparer.OrdinalIgnoreCase))
{
// Only instruction sets with are specifiable should be printed to the help text
if (instructionSet.Specifiable)
{
if (first)
{
Console.Write(arch);
Console.Write(": ");
first = false;
}
else
{
Console.Write(", ");
}
Console.Write(instructionSet.Name);
}
}
if (first) continue; // no instruction-set found for this architecture
Console.WriteLine();
}

and

Console.WriteLine(SR.InstructionSetHelp);
foreach (string arch in ValidArchitectures)
{
TargetArchitecture targetArch = Helpers.GetTargetArchitecture(arch);
bool first = true;
foreach (var instructionSet in Internal.JitInterface.InstructionSetFlags.ArchitectureToValidInstructionSets(targetArch).DistinctBy((instructionSet) => instructionSet.Name, StringComparer.OrdinalIgnoreCase))
{
// Only instruction sets with are specifiable should be printed to the help text
if (instructionSet.Specifiable)
{
if (first)
{
Console.Write(arch);
Console.Write(": ");
first = false;
}
else
{
Console.Write(", ");
}
Console.Write(instructionSet.Name);
}
}
if (first) continue; // no instruction-set found for this architecture
Console.WriteLine();
}

Add more info in --instruction-set option of crossgen2 help output.
@viewizard
Copy link
Member Author

It would be nice to also update the help text with native and optimistic options (ilc --help, crossgen2 --help):

Good point, thanks! Added.

@am11 am11 added area-crossgen2-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 31, 2025
@jkotas
Copy link
Member

jkotas commented Nov 1, 2025

/ba-g infrastructure timeouts

@jkotas jkotas merged commit b71517e into dotnet:main Nov 1, 2025
92 of 97 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-crossgen2-coreclr community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants