[PATCH] D27551: [DebugInfo] Add regression test for __fp16, float, and double constants.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 14:27:09 PST 2016


dblaikie added a comment.

Does this need the test coverage if it's just "hey, LLVM can splat these bytes out into DWARF even if we (independently) say the type of those bytes is a float"? The code that emits the bytes is presumably independent of the code that emits the type description, so testing the combination seems unnecessary to me.


https://reviews.llvm.org/D27551





More information about the llvm-commits mailing list