Avoid a copy of ExpressionEvaluator.

[wujingyue]

No description provided.

[wujingyue]

!test

[github-actions]

Description

• Changed ExpressionEvaluator to const ExpressionEvaluator& in tensor_metadata.cpp

• Updated function signature in tensor_metadata.h

---

Changes walkthrough 📝

Relevant files

Enhancement

tensor_metadata.cpp Use const reference for ExpressionEvaluator                            csrc/tensor_metadata.cpp Changed ExpressionEvaluator& to const ExpressionEvaluator& in ForwardTraverseFromLogicalToAlloc constructor Changed ExpressionEvaluator& to const ExpressionEvaluator& in BackwardTraverseFromLogicalToAlloc constructor Changed ExpressionEvaluator to const ExpressionEvaluator& in inferAndValidateAllocationSizesAndStrides function +5/-5      tensor_metadata.h Update function signature for ExpressionEvaluator                csrc/tensor_metadata.h Changed ExpressionEvaluator to const ExpressionEvaluator& in inferAndValidateAllocationSizesAndStrides function +1/-1

---

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests

⚡ Recommended focus areas for review Const Reference The change from ExpressionEvaluator& to const ExpressionEvaluator& should be validated to ensure that no modifications are attempted on the ExpressionEvaluator object within the methods. const ExpressionEvaluator& ee_; Const Reference The change from ExpressionEvaluator& to const ExpressionEvaluator& should be validated to ensure that no modifications are attempted on the ExpressionEvaluator object within the methods. const ExpressionEvaluator& ee_; Const Reference The change from ExpressionEvaluator& to const ExpressionEvaluator& should be validated to ensure that no modifications are attempted on the ExpressionEvaluator object within the methods. const ExpressionEvaluator& ee, Const Reference The change from ExpressionEvaluator to const ExpressionEvaluator& should be validated to ensure that no modifications are attempted on the ExpressionEvaluator object within the methods. const ExpressionEvaluator& ee) { Const Reference The change from ExpressionEvaluator to const ExpressionEvaluator& should be validated to ensure that no modifications are attempted on the ExpressionEvaluator object within the methods. const ExpressionEvaluator& ee);