[all-commits] [llvm/llvm-project] aa6512: AArch64: Use reportFatalUsageError for unsupported...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/aarch64/reportFatalUsageError-unsupported-calling-convention
  Home:   https://github.com/llvm/llvm-project
  Commit: aa6512edbcef8ccc3350544565d1602bc3f31e93
      https://github.com/llvm/llvm-project/commit/aa6512edbcef8ccc3350544565d1602bc3f31e93
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-06-19 (Thu, 19 Jun 2025)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    A llvm/test/CodeGen/AArch64/unsupported-cc-call.ll
    A llvm/test/CodeGen/AArch64/unsupported-cc-func.ll

  Log Message:
  -----------
  AArch64: Use reportFatalUsageError for unsupported calling conv

This probably should emit a DiagnosticInfoUnsupported and use the
default calling convention instead, but then we would need to pass
in the context.

Also move where CCAssignFnForCall is called. It was unnecessarily
called for each argument, so the error wouldn't trigger for functions
with 0 arguments.

This only ensures the error occurs for functions defined with the
calling convention. The error is still missed for outgoing calls
with no arguments. The lowering logic here is convoluted, calling
CCAssignFnForCall for each argument and it does not mirror
LowerFormalArguments so I'm not sure what's going on here.



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