Use clang instead of clang_cc1 like other C tests.

Move DetermineParentDecl to determineParentRecord in APIVisitor as a private method.

In D158031#4603363, @dang wrote:

Did this change not affect ordering of the symbols in the generated JSON? Looks pretty good to me otherwise!

Formatting changes, keep GlobalFunction has templates together and revert extraneous formatting

Fix formatting for introduced declarations in DeclarationFragments.h

Revert extraneous formatting in API.h

Note that this doesn't handle anonymous namespaces. Similar problems arise with anonymous classes/structs, being worked on.

Remove unnecessary line break.

Fix passing Decl's template, introduced from parent updates

Update to use the new Template class constructor instead of old getFragments.

Update Spec naming

Forgot Spec instance in SerializerBase

Update instances of Spec to Specialization

Also rename traverseSpec* in SerializerBase.h

Update all instances of Spec to Specialization

We didn't handle inheriting from a template parameter, but now we do by serializing the relationship. The parameter does not produce a target USR so it falls back to its name.

Address review feedback

Fix arbitrary include changes

Get rid of unnecessary function for checking if global var

Fix template arg identification

Add brackets for SerializerBase visit methods

Refactor template arg name deduction to get as string for FragmentKind::typeIdentifier

Abstract the name deduction for generic template args to its own function. It's very helpful in other cases.

Add documentation to template argument name deduction, add missing method declarations to visitor base

Format DeclarationFragments.h, try to address buildbot fail on clang-format

Remove Parser include and remove bazel dependency introduced in 926f85db98aa.

Reintroduce fix from f4de606ef271 and use clang frontend for C++ tests.

@haowei Thank you for letting me know! That patch does seem to fix it, unfortunately it didn't fail on my end. Thanks again.

Add test for overloaded operator.

Address review feedback, add tests, and small refactors.

Move semicolon line below After fragment

There aren't any variables or fields in the tests currently, waiting on D154038 because of no semicolons in their fragments.

Add C++ class tests, fix exception spec fragments

These updated tests pass locally for me, not sure why the build test failed.

Address some review feedback

Add conversion function and overloaded operator support

Fix duplicate visitation of CXXRecordDecls by overloading WalkUpFromCXXRecordDecl

Do not include return types for constructors/destructors

Add access control serialization

Address review feedback

Really address review feedback

Address some review feedback