[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