[PATCH] Don't Promote x86_fp80 byval Pointer Arguments

Thomas Jablin tjablin at gmail.com
Wed Aug 27 13:46:41 PDT 2014


Hi Reid,
I have prepared a new version of the patch to address your feedback. The patch includes new tests, so arguments are only promoted if they have no padding or if we can prove the padding bytes are not accessed. In either case, it is safe to pass the elements by value without worrying about complications due to padding bytes. I have added a data layout field to the tail.ll test case. On a platform where i32 is 64-bit aligned, the promotion tested by tail.ll is unsound.
Tom

http://reviews.llvm.org/D5057

Files:
  lib/Transforms/IPO/ArgumentPromotion.cpp
  test/Transforms/ArgumentPromotion/fp80.ll
  test/Transforms/ArgumentPromotion/tail.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5057.12998.patch
Type: text/x-patch
Size: 6566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140827/21a6d5cf/attachment.bin>


More information about the llvm-commits mailing list