[all-commits] [llvm/llvm-project] 2dc3b5: [HLSL] Apply NoRecurse attrib to all HLSL function...

[Greg Roth via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2dc3b509879518340b991733bfde5c7a4becd559
      https://github.com/llvm/llvm-project/commit/2dc3b509879518340b991733bfde5c7a4becd559
  Author: Greg Roth <grroth at microsoft.com>
  Date:   2024-08-29 (Thu, 29 Aug 2024)

  Changed paths:
    M clang/lib/CodeGen/CodeGenFunction.cpp
    A clang/test/CodeGenHLSL/implicit-norecurse-attrib.hlsl
    M clang/test/CodeGenHLSL/this-assignment-overload.hlsl
    M clang/test/CodeGenHLSL/this-assignment.hlsl

  Log Message:
  -----------
  [HLSL] Apply NoRecurse attrib to all HLSL functions (#105907)

Previously, functions named "main" got the NoRecurse attribute
consistent with the behavior of C++, which HLSL largely follows.
However, standard recursion is not allowed in HLSL, so all functions
should really have this attribute. This doesn't prevent recursion, but
rather signals that these functions aren't expected to recurse.

Practically, this was done so that entry point functions named "main"
would have all have the same attributes as otherwise identical entry
points with other names.

This required small changes to the this assignment tests because they no
longer generate so many attribute sets since more of them match.

related to #105244
but done to simplify testing for #89806



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