[PATCH] D56658: [MSP430] Add msp430 toochain

[Nathan Chancellor via Phabricator via cfe-commits]

nathanchance 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(),
----------------
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


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56658/new/

https://reviews.llvm.org/D56658