[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

[Chandler Carruth]

Yes it is. THere is more than one such edge too.


On Wed, Jul 25, 2012 at 1:40 PM, Benjamin Kramer <benny.kra at gmail.com>wrote:

>
> 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
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120725/964d4d70/attachment.html>