[all-commits] [llvm/llvm-project] 6d8901: [DXIL] Set DXIL Version in DXIL target triple base...

[S. Bharadwaj Yadavalli via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6d8901488f160dd92aea5b98fcc21c7fa7c1cbe6
      https://github.com/llvm/llvm-project/commit/6d8901488f160dd92aea5b98fcc21c7fa7c1cbe6
  Author: S. Bharadwaj Yadavalli <Bharadwaj.Yadavalli at microsoft.com>
  Date:   2024-05-08 (Wed, 08 May 2024)

  Changed paths:
    M clang/lib/Basic/Targets.cpp
    M clang/lib/Driver/ToolChains/HLSL.cpp
    M clang/test/CodeGenHLSL/basic-target.c
    M clang/test/Driver/dxc_dxv_path.hlsl
    M clang/test/Options/enable_16bit_types_validation.hlsl
    M clang/unittests/Driver/DXCModeTest.cpp
    M llvm/include/llvm/TargetParser/Triple.h
    M llvm/lib/IR/Verifier.cpp
    M llvm/lib/TargetParser/Triple.cpp
    M llvm/unittests/TargetParser/TripleTest.cpp

  Log Message:
  -----------
  [DXIL] Set DXIL Version in DXIL target triple based on shader model version (#91407)

This change set restores commit 080978dd2067d0c9ea7e229aa7696c2480d89ef1 that was reverted to address ASAN
failures and includes a fix for the ASAN failures. 

Following is the description of the change:

An earlier commit provided a way to decouple DXIL version from Shader
Model version by representing the DXIL version as `SubArch` in the DXIL
Target Triple and adding corresponding valid DXIL Arch types.
    
This change constructs DXIL target triple with DXIL version that is
deduced from Shader Model version specified in the following scenarios:
  
1. When compilation target profile is specified:
    For e.g., DXIL target triple `dxilv1.8-unknown-shader6.8-library` is
    constructed when `-T lib_6_8` is specified.
2. When DXIL target triple without DXIL version is specified:
    For e.g., DXIL target triple `dxilv1.8-pc-shadermodel6.8-library` is
    constructed when `-mtriple=dxil-pc-shadermodel6.8-library` is specified.
    
Updated relevant HLSL tests that check for target triple.



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