[llvm] r312031 - [FunctionImport] Avoid unused variable warnings in Release builds

[Teresa Johnson via llvm-commits]

Thanks! Teresa

On Tue, Aug 29, 2017 at 1:24 PM, Benjamin Kramer via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: d0k
> Date: Tue Aug 29 13:24:39 2017
> New Revision: 312031
>
> URL: http://llvm.org/viewvc/llvm-project?rev=312031&view=rev
> Log:
> [FunctionImport] Avoid unused variable warnings in Release builds
>
> Just skip the entire block in NDEBUG. No functionality change intended.
>
> Modified:
>     llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp
>
> Modified: llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/
> Transforms/IPO/FunctionImport.cpp?rev=312031&r1=312030&r2=312031&view=diff
> ============================================================
> ==================
> --- llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp (original)
> +++ llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp Tue Aug 29 13:24:39
> 2017
> @@ -726,6 +726,7 @@ Expected<bool> FunctionImporter::importF
>          GlobalsToImport.insert(&GV);
>        }
>      }
> +#ifndef NDEBUG
>      for (GlobalAlias &GA : SrcModule->aliases()) {
>        if (!GA.hasName())
>          continue;
> @@ -735,6 +736,7 @@ Expected<bool> FunctionImporter::importF
>                     << " " << GA.getName() << " from "
>                     << SrcModule->getSourceFileName() << "\n");
>      }
> +#endif
>
>      // Upgrade debug info after we're done materializing all the globals
> and we
>      // have loaded all the required metadata!
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>



-- 
Teresa Johnson |  Software Engineer |  tejohnson at google.com |  408-460-2413
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170829/38851e8c/attachment.html>