[cfe-commits] PATCH: Minor refactoring: add CodeGenFunction::maybeInitializeDebugInfo()

Eric Christopher echristo at gmail.com
Mon Oct 15 14:21:30 PDT 2012


On Mon, Oct 15, 2012 at 8:46 AM, Alexey Samsonov <samsonov at google.com> wrote:
> Hi,
>
> This small patch transforms occurrences of pattern
> if (CGM.getModuleDebugInfo())
>   DebugInfo = CGM.getModuleDebugInfo()
> into a call
>   maybeInitializeDebugInfo();
>
> This is a simplification for a possible future fix of PR13942.
>
> Code review: https://codereview.appspot.com/6699043/
>

No objections, mind adding some sort of comment before the use? I.e.
"Initialize debug info if needed." or some such?

-eric



More information about the cfe-commits mailing list