[PATCH] D42784: [ARM] Allow Half types in ConstantPool

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 08:00:35 PST 2018


SjoerdMeijer updated this revision to Diff 133865.
SjoerdMeijer added a comment.

It was indeed feeling when there e.g. was an existing 4-byte entry, which was
then followed by a 2-byte entry; the two new ARM and Thumb test cases
showed this. As suggested, we are now always emitting an alignment directive
(I've removed that if-statement).


https://reviews.llvm.org/D42784

Files:
  lib/Target/ARM/ARMConstantIslandPass.cpp
  test/CodeGen/ARM/fp16-litpool.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42784.133865.patch
Type: text/x-patch
Size: 5269 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180212/7f73a60f/attachment.bin>


More information about the llvm-commits mailing list