SWDEV-246687 - Do not use std::vector reference as class member cuMask_

The current implementation creates default reference in the stack and assigns it to class member cuMasks_, so whenever the content of the stack changes, cuMask_ would change.

Change-Id: Iefab63c335d504b83c4ae90bd34ae76c6afb8f3c


[ROCm/clr commit: 8ef5da00c7]
This commit is contained in:
Jason Tang committed 2020-08-05 16:57:36 -04:00
1 parent 7f5db8ed81
commit e1b0edf35c
3 files changed
+11 -11

No files matched your search

@@ -131,14 +131,14 @@ class CommandQueue : public RuntimeObject {
context_(context),
cuMask_(cuMask){}
Properties properties_; //!< Queue properties
uint rtCUs_; //!< The number of used RT compute units
Priority priority_; //!< Queue priority
Monitor queueLock_; //!< Lock protecting the queue
Monitor lastCmdLock_; //!< Lock protecting the last queued command
Device& device_; //!< The device
SharedReference<Context> context_; //!< The context of this command queue
const std::vector<uint32_t>& cuMask_; //!< The CU mask
Properties properties_; //!< Queue properties
uint rtCUs_; //!< The number of used RT compute units
Priority priority_; //!< Queue priority
Monitor queueLock_; //!< Lock protecting the queue
Monitor lastCmdLock_; //!< Lock protecting the last queued command
Device& device_; //!< The device
SharedReference<Context> context_; //!< The context of this command queue
const std::vector<uint32_t> cuMask_; //!< The CU mask
private:
//! Disable copy constructor