Hi Devang, nice idea.

> +  // Emit intermediate .bc file before module level optimization passes
> are run. +  if (emit_llvm_bc && flag_debug_llvm_module_opt) {

Why condition this on emit_llvm_bc?

> +    static PassManager *IntermediatePM = new PassManager();
> +    IntermediatePM->add(new TargetData(*TheTarget->getTargetData()));
> +
> +  // Emit an LLVM .bc file to the output.  This is used when passed
> +  // -emit-llvm -c to the GCC driver.

This comment is not aligned properly.  Also, it doesn't make much
sense: shouldn't it mention flag_debug_llvm_module_opt and not -c?

> +    AsmIntermediateOutFile = new OStream(*AsmIntermediateOutStream);

This doesn't seem to be used.

Best wishes,

Duncan.

_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to