Patch for http://reviews.llvm.org/D10518

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Jun 19 07:52:55 PDT 2015


invalid-data-after-module.bc is not used.

The patch has not been git-clang-formated.

It looks like there is an independent part of the patch with is just
about removing IsStreamed. I have extracted it to the attached patch.

It has a small but noticeable impact on performance. I timed running
"opt -disable-output -disable-verify" on a ltoed clang. It goes from

14.752845231 seconds time elapsed
    ( +-  0.16% )

to

15.012463721 seconds time elapsed
    ( +-  0.11% )

It is small enough that I think it is worth the dropping of code path
differences between the two cases.

Do you agree with the attached patch? If so, please commit it first
and rebase the rest of your patch.

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: text/x-patch
Size: 3245 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150619/badee959/attachment.bin>


More information about the llvm-commits mailing list