[llvm-commits] [llvm] r160751 - in /llvm/trunk: include/llvm/Analysis/MemoryBuiltins.h lib/Analysis/MemoryBuiltins.cpp lib/Transforms/Instrumentation/BoundsChecking.cpp test/Instrumentation/BoundsChecking/strings.ll

[Benjamin Kramer]

On 25.07.2012, at 22:24, Nuno Lopes <nunoplopes at sapo.pt> wrote:

> Quoting Duncan Sands <baldrick at free.fr>:
> 
>> Hi Nuno,
>> 
>>> original commit msg:
>>> MemoryBuiltins: add support to determine the size of strdup'ed  
>>> non-constant strings
>> 
>> the dragonegg bots don't use cmake but they broke too:
>> 
>> MemoryBuiltins.cpp:(.text+0x40dc): undefined reference to
>> `llvm::EmitStrNLen(llvm::Value*, llvm::Value*, llvm::IRBuilder<true,
>> llvm::ConstantFolder, llvm::IRBuilderDefaultInserter<true> >&,  
>> llvm::TargetData
>> const*, llvm::TargetLibraryInfo const*)'
>> MemoryBuiltins.cpp:(.text+0x4207): undefined reference to
>> `llvm::EmitStrLen(llvm::Value*, llvm::IRBuilder<true, llvm::ConstantFolder,
>> llvm::IRBuilderDefaultInserter<true> >&, llvm::TargetData const*,
>> llvm::TargetLibraryInfo const*)'
> 
> Interesting.. I guess I need to try to build it on another machine.

Isn't using lib/Transforms/Utils from lib/Analysis a layering violation (cyclic dependency)?

- Ben