[PATCH] D54911: [compiler-rt][builtins][PowerPC] Add ___fixunstfti builtin compiler-rt method support for PowerPC

[Amy Kwan via Phabricator via llvm-commits]

amyk updated this revision to Diff 179115.
amyk edited the summary of this revision.
amyk added a comment.

Updated this patch to address:

- Updated comments that are more descriptive
- Renamed variables
- Removed the macro originally used to group up common code. Since the code that seems duplicated is not entirely identical, the code will remain duplicated as utilizing a macro still requires an if-condition to control the conversion depending if we are converting and high double or a low double

This revision has been tested using Masoud's test harness. There are no changes or errors uncovered in the test harness with this updated revision.


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

https://reviews.llvm.org/D54911

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54911.179115.patch
Type: text/x-patch
Size: 78318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181220/d913e47f/attachment.bin>