[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