[all-commits] [llvm/llvm-project] 38cf35: [Inline] Add test for #67054 (NFC)

[Nikita Popov via All-commits]

  Branch: refs/heads/release/18.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 38cf35dee880e9e8545e7c2997201ae28f3a6738
      https://github.com/llvm/llvm-project/commit/38cf35dee880e9e8545e7c2997201ae28f3a6738
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2024-03-13 (Wed, 13 Mar 2024)

  Changed paths:
    M llvm/test/Transforms/Inline/X86/call-abi-compatibility.ll

  Log Message:
  -----------
  [Inline] Add test for #67054 (NFC)

(cherry picked from commit cad6ad2759a782c48193f83886488dacc9f330e3)


  Commit: 8c6015db5912dee1cce0e900b6abe5735be09b83
      https://github.com/llvm/llvm-project/commit/8c6015db5912dee1cce0e900b6abe5735be09b83
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2024-03-13 (Wed, 13 Mar 2024)

  Changed paths:
    M llvm/lib/Target/X86/X86TargetTransformInfo.cpp
    M llvm/test/Transforms/Inline/X86/call-abi-compatibility.ll

  Log Message:
  -----------
  [X86][Inline] Skip inline asm in inlining target feature check (#83820)

When inlining across functions with different target features, we
perform roughly two checks:
 1. The caller features must be a superset of the callee features.
2. Calls in the callee cannot use types where the target features would
change the call ABI (e.g. by changing whether something is passed in a
zmm or two ymm registers). The latter check is very crude right now.

The latter check currently also catches inline asm "calls". I believe
that inline asm should be excluded from this check, as it is independent
from the usual call ABI, and instead governed by the inline asm
constraint string.

Fixes https://github.com/llvm/llvm-project/issues/67054.

(cherry picked from commit e84182af919d136d74b75ded4d599b38fb47dfb0)


Compare: https://github.com/llvm/llvm-project/compare/c7eb919d2cbe...8c6015db5912

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