asl added inline comments.
================
Comment at: cfe/trunk/lib/CodeGen/CodeGenModule.cpp:141
+ (!CodeGenOpts.RelaxedAliasing && CodeGenOpts.OptimizationLevel > 0)) {
+ fprintf(stderr, "TBAA enabled\n");
TBAA.reset(new CodeGenTBAA(Context, TheModule, CodeGenOpts, getLangOpts(),
----------------
nathanchance wrote:
> asl wrote:
> > nathanchance wrote:
> > > I'm just curious, was committing this change intentional? Seems like
> > > something for debugging purposes, rather than a useful diagnostic, since
> > > it doesn't appear to be related to the rest of the patch (as far as I can
> > > tell).
> > >
> > > I see it quite a bit on a Linux kernel build:
> > > https://gist.github.com/nathanchance/0b2d3b199996d5b30396038d86f5cba3
> > Oh... Sorry, maybe unclean tree or something like this. Will fix!
> Thanks for the quick reply, looks like Peter Collingbourne beat you to it :)
> https://reviews.llvm.org/rC351241
Pete already fixed this in r351241.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56658/new/
https://reviews.llvm.org/D56658
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
