[PATCH] Fixing a heisenbug where the memory dependence analysis behaves differently with and without -g

Manman Ren manman.ren at gmail.com
Wed Nov 13 13:49:05 PST 2013


Hi Yunzhong,

A few nits:

In the file name "inst_limits.ll", we prefer "-" instead of "_".

+      // Debug intrinsics don't (and can't) cause dependences.
--> dependencies

I agree with David that you should only need one dbg.value intrinsic to
show the problem.

Thanks for working on this,
Manman



On Wed, Nov 13, 2013 at 12:54 PM, Yunzhong Gao <
Yunzhong_Gao at playstation.sony.com> wrote:

>   I came up with the following test case.
>
> http://llvm-reviews.chandlerc.com/D2141
>
> CHANGE SINCE LAST DIFF
>   http://llvm-reviews.chandlerc.com/D2141?vs=5480&id=5516#toc
>
> Files:
>   lib/Analysis/MemoryDependenceAnalysis.cpp
>   test/Transforms/DeadStoreElimination/inst_limits.ll
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131113/55cdc150/attachment.html>


More information about the llvm-commits mailing list