[PATCH] D56658: [MSP430] Add msp430 toochain

Tue Jan 15 13:27:00 PST 2019

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:
> 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!

Repository:
  rL LLVM

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

https://reviews.llvm.org/D56658