[llvm-commits] emit-llvm

Rafael Espindola espindola at google.com
Thu Jun 3 14:15:23 PDT 2010


On 3 June 2010 16:49, Nick Lewycky <nlewycky at google.com> wrote:
>  namespace options {
> +  enum generate_bc { BC_NO, BC_ALSO, BC_ONLY};
>    static bool generate_api_file = false;
> Please fix the whitespace; you're missing a space after BC_ONLY before }

Done.

> +    if (options::generate_bc_file == options::BC_ONLY)
> +      exit(0);
> Eww!! I assume there's no other way to implement this, or else you would've
> done that instead.
> Some day, we should teach gold to let a plugin generate the output. It could
> be useful for things like ld -r on bitcode files, for example. Until then,
> this will have to do.

ld -r -plugin-opt=emit-llvm might actually work now :-)
What kind of API would you like from gold? An exit callback?

> Nick

Cheers,
-- 
Rafael Ávila de Espíndola




More information about the llvm-commits mailing list