[PATCH] D49746: [WIP][LV][DebugInfo] Set DL to the middle block Icmp instruction

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 31 05:25:49 PDT 2018


xbolva00 added a comment.

In https://reviews.llvm.org/D49746#1177945, @gramanas wrote:

> http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/18826/steps/ninja%20check%201/logs/stdio
>
> This results in many ICmp Instructions to have completely wrong `!dbg` attached, since it points to a different subprogram.
>
> I reverted the commit in https://reviews.llvm.org/rL338109, but I don't understand why this happens.
>
> Also I think the change of `CmpN` from `Value` to `CmpInst` triggered these warnings: http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/18826/steps/build%20stage%201/logs/warnings%20%284%29


That -Woverloaded-virtual warnings are not related to your patch :)


Repository:
  rL LLVM

https://reviews.llvm.org/D49746





More information about the llvm-commits mailing list