[llvm] r191343 - Move LTO support library to a component, allowing it to be tested

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Sep 27 16:22:29 PDT 2013


On 27 September 2013 16:37, Tom Roeder <tmroeder at google.com> wrote:
> Hi Richard,
>
> Thanks for catching that. The LTO library code was including
> llvm/CodeGen/CommandFlags.h, which was also getting included in
> binaries like llc, hence the multiple definitions.

Where? I only see:

$ git grep CommandFlags.h
include/llvm/CodeGen/CommandFlags.h://===-- CommandFlags.h - Command
Line Flags Interface -----------*- C++ -*-===//
tools/llc/llc.cpp:#include "llvm/CodeGen/CommandFlags.h"
tools/opt/opt.cpp:#include "llvm/CodeGen/CommandFlags.h"

> The attached patch fixes this by refactoring the flags out of lib/LTO
> and up into the tools (llvm-lto and lto) that use them.
>\
> Tom


Cheers,
Rafael



More information about the llvm-commits mailing list