[PATCH] D53390: [DebugInfo][Dexter] Unreachable line stepped onto after SimplifyCFG.
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 29 08:42:10 PDT 2018
aprantl added inline comments.
================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:1393
+ // location (DILocation); instead use the value from the reference insertion
+ // point.
+ NT->setDebugLoc(InsertPt->getDebugLoc());
----------------
This comment explains *what* the code is doing but not *why*. Could you explain why this is being done in the comment?
================
Comment at: test/CodeGen/X86/pr39187.ll:62
+ %beards.0 = phi i32 [ 8, %if.then ], [ 4, %if.else ], !dbg !25
+ call void @llvm.dbg.value(metadata i32 %beards.0, metadata !14, metadata !DIExpression()), !dbg !20
+ ret i32 %beards.0
----------------
Since this test is only about DILocations, stripping out the dbg.value would make the the metadata a lot shorter.
Repository:
rL LLVM
https://reviews.llvm.org/D53390
More information about the llvm-commits
mailing list