[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