search for: coffdump

tools/llvm-objdump/COFFDump.cpp has two loops that advance the loop iterator using "I = ++I" instead of the usual "++I". For example: import_directory_iterator I = Obj->import_directory_begin(); import_directory_iterator E = Obj->import_directory_end(); if (I == E) return; outs() <&l...

Oops, meant to send this to the mailing list instead of to Reid privately. (Why cc the mailing list instead of just sending to the mailing list?) In article <CACs=tyJ6zaHeiS0eNhBkdcZE--JY4k7yH9_P1yFbGqod6uymMw at mail.gmail.com>, Reid Kleckner <rnk at google.com> writes: > Looks like a bug. This can probably be simplified with C++11. > > On Fri, Apr 4, 2014 at 10:39

On Mon, Apr 7, 2014 at 12:09 PM, Joerg Sonnenberger <joerg at britannica.bec.de > wrote: > On Mon, Apr 07, 2014 at 08:38:58AM -0600, Richard wrote: > > Oops, meant to send this to the mailing list instead of to Reid > > privately. (Why cc the mailing list instead of just sending to the > > mailing list?) > > > > In article <CACs= >

...FF.h so that it doesn't clash? I know windows.h is quite invasive, so maybe it's better not try to collide with any of its weird define. If that makes sense I'm fine with doing the patch. If yes, feel free to propose an alternative naming prefix or scheme. Note that if tools depend on COFFDumper::printFileHeaders output, it might still need to print as it was before -- so the easiest choice might be to maybe just drop the IMAGE_ (anyway it's in llvm::COFF so it shouldn't matter). Is it important? Thanks, Virgile -------------- next part -------------- An HTML attachment was scru...

...? > I know windows.h is quite invasive, so maybe it's better not try to > collide with any of its weird define. > > If that makes sense I'm fine with doing the patch. > If yes, feel free to propose an alternative naming prefix or scheme. > > Note that if tools depend on COFFDumper::printFileHeaders output, it might > still need to print as it was before -- so the easiest choice might be to > maybe just drop the IMAGE_ (anyway it's in llvm::COFF so it shouldn't > matter). Is it important? > > Thanks, > Virgile > > ____________________________...

...initializers -pedantic -Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Werror=date-time -std=c++11 -O3 -Wl,-search_paths_first -Wl,-headerpad_max_install_names -L/sw/lib -Wl,-dead_strip CMakeFiles/llvm-objdump.dir/llvm-objdump.cpp.o CMakeFiles/llvm-objdump.dir/COFFDump.cpp.o CMakeFiles/llvm-objdump.dir/ELFDump.cpp.o CMakeFiles/llvm-objdump.dir/MachODump.cpp.o -o ../../bin/llvm-objdump ../../lib/libLLVM.dylib -Wl,-rpath, at executable_path/../lib -lxar eliminates the bootstrap failure. > > Kev > >> On May 24, 2016, at 10:28 AM, Jack Howarth <...

...initializers -pedantic -Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Werror=date-time -std=c++11 -O3 -Wl,-search_paths_first -Wl,-headerpad_max_install_names -L/sw/lib -Wl,-dead_strip CMakeFiles/llvm-objdump.dir/llvm-objdump.cpp.o CMakeFiles/llvm-objdump.dir/COFFDump.cpp.o CMakeFiles/llvm-objdump.dir/ELFDump.cpp.o CMakeFiles/llvm-objdump.dir/MachODump.cpp.o -o ../../bin/llvm-objdump ../../lib/libLLVMX86AsmPrinter.dylib ../../lib/libLLVMPowerPCAsmPrinter.dylib ../../lib/libLLVMARMAsmPrinter.dylib ../../lib/libLLVMX86Desc.dylib ../../lib/libLLVMPowerPCDesc.dyli...

...g -Wcovered-switch-default -Wnon-virtual-dtor >> -Wdelete-non-virtual-dtor -Werror=date-time -std=c++11 -O3 >> -Wl,-search_paths_first -Wl,-headerpad_max_install_names -L/sw/lib >> -Wl,-dead_strip CMakeFiles/llvm-objdump.dir/llvm-objdump.cpp.o >> CMakeFiles/llvm-objdump.dir/COFFDump.cpp.o >> CMakeFiles/llvm-objdump.dir/ELFDump.cpp.o >> CMakeFiles/llvm-objdump.dir/MachODump.cpp.o -o ../../bin/llvm-objdump >> ../../lib/libLLVM.dylib -Wl,-rpath, at executable_path/../lib -lxar >> >> eliminates the bootstrap failure. >> >>> >>&gt...

...ch-default -Wnon-virtual-dtor >>> -Wdelete-non-virtual-dtor -Werror=date-time -std=c++11 -O3 >>> -Wl,-search_paths_first -Wl,-headerpad_max_install_names -L/sw/lib >>> -Wl,-dead_strip CMakeFiles/llvm-objdump.dir/llvm-objdump.cpp.o >>> CMakeFiles/llvm-objdump.dir/COFFDump.cpp.o >>> CMakeFiles/llvm-objdump.dir/ELFDump.cpp.o >>> CMakeFiles/llvm-objdump.dir/MachODump.cpp.o -o ../../bin/llvm-objdump >>> ../../lib/libLLVM.dylib -Wl,-rpath, at executable_path/../lib -lxar >>> >>> eliminates the bootstrap failure. >>>...

...t; -Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor > -Wdelete-non-virtual-dtor -Werror=date-time -std=c++11 -O3 > -Wl,-search_paths_first -Wl,-headerpad_max_install_names -L/sw/lib > -Wl,-dead_strip CMakeFiles/llvm-objdump.dir/llvm-objdump.cpp.o > CMakeFiles/llvm-objdump.dir/COFFDump.cpp.o > CMakeFiles/llvm-objdump.dir/ELFDump.cpp.o > CMakeFiles/llvm-objdump.dir/MachODump.cpp.o -o ../../bin/llvm-objdump > ../../lib/libLLVM.dylib -Wl,-rpath, at executable_path/../lib -lxar > > eliminates the bootstrap failure. > >> >> Kev >> >>>...

...is quite invasive, so maybe it's better not try to >> collide with any of its weird define. >> >> If that makes sense I'm fine with doing the patch. >> If yes, feel free to propose an alternative naming prefix or scheme. >> >> Note that if tools depend on COFFDumper::printFileHeaders output, it >> might still need to print as it was before -- so the easiest choice might >> be to maybe just drop the IMAGE_ (anyway it's in llvm::COFF so it shouldn't >> matter). Is it important? >> >> Thanks, >> Virgile >> >&...

Hi Jack, Just a guess here, this may be the bug Chris helped me out with in the use of the include file xar/xar.h which is not C++ safe. I needed to wrap my include via: #ifdef HAVE_LIBXAR extern "C" { #include <xar/xar.h> } #endif I think we may need some help from Chris to track this down. I’ll bug him in a bit to see if he can help us on this. Kev > On May 24, 2016, at

On Tue, May 24, 2016 at 1:24 PM, Jack Howarth <howarth.mailing.lists at gmail.com> wrote: > On Tue, May 24, 2016 at 1:22 PM, Jack Howarth > <howarth.mailing.lists at gmail.com> wrote: >> On Tue, May 24, 2016 at 12:08 PM, Reid Kleckner <rnk at google.com> wrote: >>> Kevin Enderby added those symbol uses in r270491. It has a cmake >>> feature test, and

...e llvm-diff (without symbols) make[2]: Entering directory '/home/NIKHILREDDY/WORK/LLVM_OBJ/tools/llvm-objdump' llvm[2]: ======= Finished Linking Release+Asserts Executable lli (without symbols) make[2]: Leaving directory '/home/NIKHILREDDY/WORK/LLVM_OBJ/tools/lli' llvm[2]: Compiling COFFDump.cpp for Release+Asserts build make[2]: Entering directory '/home/NIKHILREDDY/WORK/LLVM_OBJ/tools/llvm-readobj' llvm[2]: Compiling ARMAttributeParser.cpp for Release+Asserts build llvm[2]: Linking Release+Asserts executable bugpoint (without symbols) llvm[2]: ======= Finished Linking Release...