Author: Nathan James Date: 2021-04-15T00:19:40+01:00 New Revision: 6890f302f587ca4f6712be728bd9302da597fe0c
URL: https://github.com/llvm/llvm-project/commit/6890f302f587ca4f6712be728bd9302da597fe0c DIFF: https://github.com/llvm/llvm-project/commit/6890f302f587ca4f6712be728bd9302da597fe0c.diff LOG: [AST][Introspection] Fix args not being set. This field isn't set in the constructor. Tweak its accessor to return an ArrayRef. Added: Modified: clang/include/clang/Tooling/NodeIntrospection.h Removed: ################################################################################ diff --git a/clang/include/clang/Tooling/NodeIntrospection.h b/clang/include/clang/Tooling/NodeIntrospection.h index 70bfeba8c4f9..406e17f1351a 100644 --- a/clang/include/clang/Tooling/NodeIntrospection.h +++ b/clang/include/clang/Tooling/NodeIntrospection.h @@ -37,15 +37,15 @@ class LocationCall : public llvm::ThreadSafeRefCountedBase<LocationCall> { enum LocationCallFlags { NoFlags, ReturnsPointer, IsCast }; LocationCall(SharedLocationCall on, std::string name, LocationCallFlags flags = NoFlags) - : m_flags(flags), m_on(std::move(on)), m_name(name) {} + : m_flags(flags), m_on(std::move(on)), m_name(std::move(name)) {} LocationCall(SharedLocationCall on, std::string name, - std::vector<std::string> const &args, - LocationCallFlags flags = NoFlags) - : m_flags(flags), m_on(std::move(on)), m_name(name) {} + std::vector<std::string> args, LocationCallFlags flags = NoFlags) + : m_flags(flags), m_on(std::move(on)), m_name(std::move(name)), + m_args(std::move(args)) {} LocationCall *on() const { return m_on.get(); } StringRef name() const { return m_name; } - std::vector<std::string> const &args() const { return m_args; } + ArrayRef<std::string> args() const { return m_args; } bool returnsPointer() const { return m_flags & ReturnsPointer; } bool isCast() const { return m_flags & IsCast; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
