[all-commits] [llvm/llvm-project] 0be3f1: [DirectX] only allow intrinsics defined in DXIL.td...

Farzon Lotfi via All-commits all-commits at lists.llvm.org
Tue Feb 25 13:10:03 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0be3f134c3b0bea0a3f32db55258c776caf616fb
      https://github.com/llvm/llvm-project/commit/0be3f134c3b0bea0a3f32db55258c776caf616fb
  Author: Farzon Lotfi <farzonlotfi at microsoft.com>
  Date:   2025-02-25 (Tue, 25 Feb 2025)

  Changed paths:
    M llvm/lib/Target/DirectX/DXILOpLowering.cpp
    M llvm/test/CodeGen/DirectX/clamp.ll
    M llvm/test/CodeGen/DirectX/discard.ll
    A llvm/test/CodeGen/DirectX/unsupported_intrinsic.ll

  Log Message:
  -----------
  [DirectX] only allow intrinsics defined in DXIL.td (#128613)

Fixes #128071
The current behavior lets intrinsics that don't map to a DXILOP slip
through. Nothing catches this until we hit the DXIL validator. This
change fails earlier so we don't encode invalid llvm intrinsics that can
slip through because of clang builtins like `__builtin_reduce_and`
example:
https://hlsl.godbolt.org/z/13rPj18vn



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


More information about the All-commits mailing list