[PATCH] D15798: Fix for Bug 24852 (crash with -debug -instcombine)

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 10:00:45 PST 2016


This is what I was fearing, it needs a real metadata attached.
Replacing the setDebugLoc with 

  Add->setMetadata(
      "", MDNode::get(Ctx, {ConstantAsMetadata::get(ConstantInt::get(Ty, 1))}));

seems to do the trick though.

— 
Mehdi


> On Jan 7, 2016, at 9:16 AM, Than McIntosh <thanm at google.com> wrote:
> 
> thanm added a comment.
> 
> Hmm, I think I may have jumped the gun-- the reduced test case doesn't actually trigger the crash. I'll go back to my original code...
> 
> 
> http://reviews.llvm.org/D15798
> 
> 
> 



More information about the llvm-commits mailing list