[PATCH] Proposed fix for PR23076 (conditional branch debug line info)
David Blaikie
dblaikie at gmail.com
Fri Apr 10 09:20:35 PDT 2015
Neither of the test case updates seem to relate to the example given in the bug/patch description - perhaps you need a test case for that?
Also, the test case updates seem to be incorrect - those branches are for the branch to/over the second operand to the && and || and should be attributed to the && and || (lines 2001 and 2101) not to the same line as the first expression, I think?
http://reviews.llvm.org/D8822
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list