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