[PATCH] D52715: [clangd] Dex iterator printer shows query structure, not iterator state.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 2 02:17:17 PDT 2018


ioeric accepted this revision.
ioeric added inline comments.
This revision is now accepted and ready to land.


================
Comment at: unittests/clangd/DexTests.cpp:239
+  auto Tree = createLimit(createAnd(move(I1), move(I2)), 10);
+  EXPECT_EQ(llvm::to_string(*Tree), "(LIMIT 10 (& [1 3 5] T=L2))");
 }
----------------
nit: it's not obvious in this test case that `[1 3 5]` and `T=L2` are both posting lists.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52715





More information about the cfe-commits mailing list