Change the GetLocations API to take instances by const-ref. This class
is copyable. APIs return copies, so the API taking a pointer was
needlessly hard to use.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
clang/lib/Tooling/DumpTool/generate_cxx_src_locs.py | ||
---|---|---|
59 | Are there more Clades that require const ref instead of pointer, it may be wise to create a set of these and re use that throughout RefClades = { "TypeLoc", "NestedNameSpecifierLoc" } ... InstanceDecoration = '&' if CladeName in RefClades else '*' | |
clang/unittests/Introspection/IntrospectionTest.cpp | ||
379–393 |
clang/lib/Tooling/DumpTool/generate_cxx_src_locs.py | ||
---|---|---|
59 | Split into https://reviews.llvm.org/D100720 |
clang/unittests/Introspection/IntrospectionTest.cpp | ||
---|---|---|
292 | After https://reviews.llvm.org/D100720 is in, I'll push an update to fix this. |
clang/unittests/Introspection/IntrospectionTest.cpp | ||
---|---|---|
292 | My guess is that googletest converts the macro to a string which is too long. |
Are there more Clades that require const ref instead of pointer, it may be wise to create a set of these and re use that throughout