[cfe-commits] r93389 - /cfe/trunk/lib/CodeGen/CGDebugInfo.cpp

[Chris Lattner]

On Jan 13, 2010, at 4:48 PM, Devang Patel wrote:

> Author: dpatel
> Date: Wed Jan 13 18:48:09 2010
> New Revision: 93389
>
> URL: http://llvm.org/viewvc/llvm-project?rev=93389&view=rev
> Log:
> Because CurLoc is the current source location as far as CGDebugInfo  
> is concerned. It is expected that this is set (usually left bracket  
> location of function body compound statement) before  
> EmitfunctionStart() is used.

Please add a comment to the code.

-Chris

>
> Modified:
>    cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
>
> Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDebugInfo.cpp?rev=93389&r1=93388&r2=93389&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- cfe/trunk/lib/CodeGen/CGDebugInfo.cpp (original)
> +++ cfe/trunk/lib/CodeGen/CGDebugInfo.cpp Wed Jan 13 18:48:09 2010
> @@ -1014,7 +1014,6 @@
>     LinkageName = Name;
>   }
>
> -  // FIXME: Why is this using CurLoc???
>   llvm::DICompileUnit Unit = getOrCreateCompileUnit(CurLoc);
>   SourceManager &SM = CGM.getContext().getSourceManager();
>   unsigned LineNo = SM.getPresumedLoc(CurLoc).getLine();
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits