[PATCH] D138392: clang/HIP: Fix broken implementations of __make_mantissa* functions

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 20 20:43:49 PST 2022


arsenm created this revision.
arsenm added reviewers: yaxunl, JonChesterfield, saiislam, scchan, AlexVlx, b-sumner.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.

The optimizer was folding the entire function to return 0. This
meant to be checking the character content of the pointer is the
string terminator, not null.

      

Not sure if the null checks are necessary. I'm not sure what
the spec is for these, or why these would even be exposed.


https://reviews.llvm.org/D138392

Files:
  clang/lib/Headers/__clang_hip_math.h
  clang/test/Headers/__clang_hip_math.hip

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138392.476797.patch
Type: text/x-patch
Size: 35448 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221121/ba4c4429/attachment-0001.bin>


More information about the cfe-commits mailing list