[all-commits] [llvm/llvm-project] 5ba57a: [HLSL] Update Sema Checking Diagnostics for builti...

[Sarah Spall via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5ba57a81f213f17547be11e199888d8d156ececc
      https://github.com/llvm/llvm-project/commit/5ba57a81f213f17547be11e199888d8d156ececc
  Author: Sarah Spall <sarahspall at microsoft.com>
  Date:   2025-05-21 (Wed, 21 May 2025)

  Changed paths:
    M clang/include/clang/Basic/Builtins.td
    M clang/include/clang/Basic/DiagnosticSemaKinds.td
    M clang/lib/CodeGen/CGHLSLBuiltins.cpp
    M clang/lib/Headers/hlsl/hlsl_alias_intrinsics.h
    M clang/lib/Sema/SemaHLSL.cpp
    M clang/test/SemaHLSL/BuiltIns/AddUint64-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/asdouble-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/clamp-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/cross-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/degrees-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/dot-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/frac-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/half-float-only-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/half-float-only-errors2.hlsl
    M clang/test/SemaHLSL/BuiltIns/isinf-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/lerp-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/logical-operator-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/mad-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/normalize-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/radians-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/rcp-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/reversebits-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/rsqrt-errors.hlsl
    M clang/test/SemaHLSL/BuiltIns/step-errors.hlsl

  Log Message:
  -----------
  [HLSL] Update Sema Checking Diagnostics for builtins (#138429)

Update how Sema Checking is done for HLSL builtins to allow for better
error messages, mainly using 'err_builtin_invalid_arg_type'.
Try to follow the formula outlined in issue #134721 
Closes #134721



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