[Lldb-commits] [PATCH] D79563: [lldb/test] Make "inline" tests handle multiple statements at the same location

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu May 7 05:37:58 PDT 2020


teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.

Will this also fix these problems?

  int i = 0;
  //% code_that_is_executed
  
  //% that that isn't executed as it's behind an empty line :(

I think disabling the check is fine. You can also just change the `'= 14'` to a `'= 11'`. I originally only intended to capture the current behavior and assumed we prefer local variables, but the agreement back then was that we don't know what is the correct behavior in this situation.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79563/new/

https://reviews.llvm.org/D79563





More information about the lldb-commits mailing list