[PATCH] D69193: [compiler-rt][builtins][PowerPC] Fix __fixunstfti builtin on PowerPC

Baptiste Saleil via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 13:46:06 PDT 2019


bsaleil created this revision.
bsaleil added reviewers: hfinkel, echristo, masoud.ataei, renenkel, nemanjai, lei, amyk.
Herald added subscribers: llvm-commits, Sanitizers, steven.zhang, shchenz, jsji, kbarton, dberris.
Herald added projects: Sanitizers, LLVM.

This patch fixes __fixunstfti builtin for PowerPC.
This builtin converts a long double (IBM double-double) to an unsigned 128 bit integer.

The patch allows the builtin to handle a previously unhandled case in which a negative low double may impact the result of the conversion.

@masoud.ataei and @renenkel helped me to fix this builtin and to generate additional tests to test the builtin using negative low doubles.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69193

Files:
  compiler-rt/lib/builtins/ppc/fixunstfti.c
  compiler-rt/test/builtins/Unit/ppc/fixunstfti_test.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69193.225674.patch
Type: text/x-patch
Size: 63177 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191018/f15a522f/attachment.bin>


More information about the llvm-commits mailing list