[llvm-commits] [llvm] r145180 - in /llvm/trunk: include/llvm-c/Object.h include/llvm/Object/COFF.h include/llvm/Object/MachO.h include/llvm/Object/ObjectFile.h lib/Object/COFFObjectFile.cpp lib/Object/ELFObjectFile.cpp lib/Object/MachOObjectFile.

Danil Malyshev dmalyshev at accesssoftek.com
Tue Nov 29 09:43:25 PST 2011


Hello,

> Danil, I've reverted this in r145182 because it broke tests on most of our build bots:
>
> http://lab.llvm.org:8011/builders/llvm-x86_64-linux/builds/1457
>
> It seems to trigger for all of the platforms, so please run the tests next time before you commit...
>
> I notice that you didn't update any test files with the commit... was that intentional? At the least, this needs tests to cover all of the "fixed" things you mention in your commit. I suspect it will also need updates to existing tests.


Sorry, missed the regression tests.
Tools fixed, mach-o testing added to nm-trivial-object.test and objdump-symbol-table.test.
Tested with regress tests on Ubuntu and Mingw32.
Committed as 145408, thank you!



Regards,
Danil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111129/514293c0/attachment.html>


More information about the llvm-commits mailing list