[PATCH] D31946: [legalize-types] Make softening result use a single map for replacements.

[Anton Yartsev via Phabricator via llvm-commits]

ayartsev updated this revision to Diff 96782.
ayartsev added a comment.

Great! Here ISD::FNEG caused the problem at least in two tests. Updated the patch. Hope this was the last regression.

> Please note that even if clang does not abort, like your test case show,
>  we should check if the output code actually put fp128 values in SSE registers
>  as before.

Done.
Compared the output code generated from the //soft-fp-legal-in-HW-reg.ll// before and after the patch - the code is the same.


https://reviews.llvm.org/D31946

Files:
  lib/CodeGen/SelectionDAG/LegalizeFloatTypes.cpp
  lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
  lib/CodeGen/SelectionDAG/LegalizeTypes.h
  lib/CodeGen/SelectionDAG/LegalizeTypesGeneric.cpp
  test/CodeGen/X86/soft-fp-legal-in-HW-reg.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31946.96782.patch
Type: text/x-patch
Size: 12409 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170426/0d8f9b0b/attachment.bin>