From 274cf7c6fe1c121d095acd38dd1a33214e0c8448 Mon Sep 17 00:00:00 2001 From: Dhruv Govil Date: Sat, 23 Nov 2024 14:06:17 -0800 Subject: [PATCH] An upcoming change in mainline LLVM is a little stricter about template syntax, causing OpenUSD to fail to compile. `Sdf_Children::_UpdateChildNames()` includes a template call that was in violation of ISO C standards to compile properly. ISO C++03 14.2/4: > When the name of a member template specialization appears after . or -> in a postfix-expression, or after nested-name-specifier in a qualified-id, and the postfix-expression or qualified-id explicitly depends on a template-parameter (14.6.2), the member template name must be prefixed by the keyword template. Otherwise the name is assumed to name a non-template. Explicitly adding the template keyword resolves this. Unfortunately this also gets tagged as unnecessary by older LLVM versions or other compilers, but c'est la vie. --- pxr/usd/sdf/children.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pxr/usd/sdf/children.cpp b/pxr/usd/sdf/children.cpp index 55eb4a9747..5370056366 100644 --- a/pxr/usd/sdf/children.cpp +++ b/pxr/usd/sdf/children.cpp @@ -190,7 +190,7 @@ Sdf_Children::_UpdateChildNames() const _childNamesValid = true; if (_layer) { - _childNames = _layer->GetFieldAs >( + _childNames = _layer->template GetFieldAs>( _parentPath, _childrenKey); } else { _childNames.clear();