search for: dumpus

...o build llvm tip as following: cmake -DCMAKE_CXX_FLAGS:STRING="-DLLVM_ENABLE_DUMP" -DCMAKE_BUILD_TYPE=RelWithDebInfo ../llvm After running 'make', I have got error messages like below. llvm/lib/CodeGen/MachineRegisterInfo.cpp:462:67: error: no ‘void llvm::MachineRegisterInfo::dumpUses(unsigned int) const’ member function declared in class ‘llvm::MachineRegisterInfo’ llvm/lib/CodeGen/MachineScheduler.cpp:2331:57: error: no ‘void llvm::SchedBoundary::dumpScheduledState()’ member function declared in class ‘llvm::SchedBoundary’ ... It seems the "defined(LLVM_ENABLE_DUM...

...flag. (this really should be fixed). > > In the meantime `cmake -DCMAKE_CXX_FLAGS="-DLLVM_ENABLE_DUMP"` should do the trick. Thank you. That was the flag we needed. Unfortunately, when I tried this just now, I found that the declaration of llvm::MachineRegisterInfo::dumpUses() in include/llvm/CodeGen/MachineRegisterInfo.h is missing the check against LLVM_ENABLE_DUMP. It has only #ifndef NDEBUG which causes the build to fail because the definition in lib/CodeGen/MachineRegisterInfo.cpp is guarded by this. #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)...

> On Apr 7, 2017, at 4:45 PM, Matthias Braun via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > I think the idea is to keep NDEBUG out of headers when possible. So I think this should better be something like: > > -#ifndef NDEBUG > void dumpUses(unsigned RegNo) const; > -#endif > > to be inline with various other dumpers (like MachineInstr::dump(), Pass::dump(), …) I’m fine with leaving methods there, but we need to be able to compile-out fields in structure. We already have ABI_BREAKING_CHECKS for instance to this end, the n...

...te: >> In the meantime `cmake -DCMAKE_CXX_FLAGS="-DLLVM_ENABLE_DUMP"` should >> do the trick. > > Thank you. That was the flag we needed. > > Unfortunately, when I tried this just now, I found that the > declaration of llvm::MachineRegisterInfo::dumpUses() in > include/llvm/CodeGen/MachineRegisterInfo.h is missing the check against > LLVM_ENABLE_DUMP. It has only > > #ifndef NDEBUG > > which causes the build to fail because the definition in > lib/CodeGen/MachineRegisterInfo.cpp is guarded by this. > > #if !defin...

...n via llvm-dev wrote: >>> In the meantime `cmake -DCMAKE_CXX_FLAGS="-DLLVM_ENABLE_DUMP"` should do the trick. >> Thank you. That was the flag we needed. >> Unfortunately, when I tried this just now, I found that the declaration of llvm::MachineRegisterInfo::dumpUses() in include/llvm/CodeGen/MachineRegisterInfo.h is missing the check against LLVM_ENABLE_DUMP. It has only >> #ifndef NDEBUG >> which causes the build to fail because the definition in lib/CodeGen/MachineRegisterInfo.cpp is guarded by this. >> #if !defined(NDEBUG) || defined(LL...

...:45 PM, Matthias Braun via llvm-dev <llvm- >> dev at lists.llvm.org> wrote: >>> >>> I think the idea is to keep NDEBUG out of headers when possible. So I >> think this should better be something like: >>> >>> -#ifndef NDEBUG >>> void dumpUses(unsigned RegNo) const; >>> -#endif >>> >>> to be inline with various other dumpers (like MachineInstr::dump(), >> Pass::dump(), …) >> >> I’m fine with leaving methods there, but we need to be able to compile-out >> fields in structure. > &g...

...; >>> In the meantime `cmake -DCMAKE_CXX_FLAGS="-DLLVM_ENABLE_DUMP"` should > do the trick. > >> Thank you. That was the flag we needed. > >> Unfortunately, when I tried this just now, I found that the > declaration of llvm::MachineRegisterInfo::dumpUses() in > include/llvm/CodeGen/MachineRegisterInfo.h is missing the check against > LLVM_ENABLE_DUMP. It has only > >> #ifndef NDEBUG > >> which causes the build to fail because the definition in lib/CodeGen/MachineRegisterInfo.cpp > is guarded by this. > >> #if...

...to:dev at lists.llvm.org>> wrote: >>>>> >>>>> I think the idea is to keep NDEBUG out of headers when possible. So I >>>> think this should better be something like: >>>>> >>>>> -#ifndef NDEBUG >>>>> void dumpUses(unsigned RegNo) const; >>>>> -#endif >>>>> >>>>> to be inline with various other dumpers (like MachineInstr::dump(), >>>> Pass::dump(), …) >>>> >>>> I’m fine with leaving methods there, but we need to be able to co...

...;>>>>> I think the idea is to keep NDEBUG out of headers when possible. >>>>>>> So I >>>>>> think this should better be something like: >>>>>>> >>>>>>> -#ifndef NDEBUG >>>>>>> void dumpUses(unsigned RegNo) const; >>>>>>> -#endif >>>>>>> >>>>>>> to be inline with various other dumpers (like MachineInstr::dump(), >>>>>> Pass::dump(), …) >>>>>> >>>>>> I’m fine with leavi...

> On Sep 25, 2017, at 6:03 PM, David Keaton via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > On 09/25/2017 02:53 PM, Matthias Braun via llvm-dev wrote: >> The dump methods can be included in the release builds anyway by enabling LLVM_ENABLE_DUMP. > > Is there a way to pass LLVM_ENABLE_DUMP into cmake from outside without changing any of the configuration files?

On 09/25/2017 02:53 PM, Matthias Braun via llvm-dev wrote: > The dump methods can be included in the release builds anyway by > enabling LLVM_ENABLE_DUMP. Is there a way to pass LLVM_ENABLE_DUMP into cmake from outside without changing any of the configuration files? With Chapel's use of LLVM, we try to avoid modifying any of the source files that come from the LLVM