[PATCH] D131718: [HLSL] Add abs library function

Chris Bieneman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 23 14:22:00 PDT 2022


beanz updated this revision to Diff 454962.
beanz added a comment.
Herald added a subscriber: MaskRay.

Updating with a fix for half type parameters. This addresses the issue I filed here: https://github.com/llvm/llvm-project/issues/57100

The issue was caused by HLSL not implying options to allow half types to be used as parameters and return types.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131718/new/

https://reviews.llvm.org/D131718

Files:
  clang/include/clang/Driver/Options.td
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/lib/Headers/hlsl/hlsl_intrinsics.h
  clang/test/CodeGenHLSL/builtins/abs.hlsl

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131718.454962.patch
Type: text/x-patch
Size: 4736 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220823/792810e2/attachment.bin>


More information about the cfe-commits mailing list