[PATCH] D20307: Follow-up patch of http://reviews.llvm.org/D19948 to handle missing profiles when simplifying CFG.

Dehao Chen via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 15:42:15 PDT 2016


The test is validated. For most cases, the prof id just shifted by one. For
test1 and test3, the changes are intended.

Dehao

On Wed, May 18, 2016 at 2:56 PM, David Li <davidxl at google.com> wrote:

> davidxl accepted this revision.
> davidxl added a comment.
> This revision is now accepted and ready to land.
>
> lgtm
>
> The code change looks fine to me -- but I have not audited the test case
> changes. Please do this at some point if not done(validate the output BP
> make sense -- there could be bugs exposed else where).
>
>
> http://reviews.llvm.org/D20307
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160518/a8176e92/attachment.html>


More information about the llvm-commits mailing list