[PATCH] D18624: [PGO] PGOFuncName meta data if PGOFuncName is different from function's raw name.

Rong Xu via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 20 15:21:52 PDT 2016


the reason for splitting the check is we don't need this check at all in
llvm instrument,-- as it's done per function. but I guess it does not
matter much to move it in to create -- it's only called once per function.





On Wed, Apr 20, 2016 at 3:04 PM, David Li <davidxl at google.com> wrote:

> davidxl added inline comments.
>
> ================
> Comment at: lib/CodeGen/CodeGenPGO.cpp:47
> @@ +46,3 @@
> +  // Create PGOFuncName meta data.
> +  if (!llvm::getPGOFuncNameMetadata(*Fn))
> +    llvm::createPGOFuncNameMetadata(*Fn);
> ----------------
> This check be folded into the creator. The creator interface name also
> needs to be changed properly (created when needed)
>
>
> http://reviews.llvm.org/D18624
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160420/808e39b3/attachment.html>


More information about the cfe-commits mailing list