[PATCH] Fix LTO handling of module-level assembly (Bug 14152)

Tom Roeder tmroeder at google.com
Fri Sep 13 16:54:41 PDT 2013


Here is a pair of patches that fix and test
http://llvm.org/bugs/show_bug.cgi?id=14512. The fix applies to LLVM,
and I have currently put the test in the clang test suite. The test is
a minor tweak to the test proposed in the bug. Please take a look.

The problem in this case is that the RecordStreamer implementation of
MCStreamer in the LTO module doesn't implement EmitCFIEndProcImpl, and
the default implementation in MCStreamer doesn't finish the frame.

This patch touches the LTO code in tools/lto, but the test uses clang.
Is this the right way to do it? Is there a better location for LTO
tests?

Thanks,

Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lto_unfinished_frame.patch
Type: application/octet-stream
Size: 575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130913/c5b68929/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lto_unfinished_frame_test.patch
Type: application/octet-stream
Size: 467 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130913/c5b68929/attachment-0001.obj>


More information about the llvm-commits mailing list