zbtzbtzbt commented on a change in pull request #8292:
URL: https://github.com/apache/incubator-doris/pull/8292#discussion_r818268350



##########
File path: be/src/vec/columns/column.h
##########
@@ -174,7 +174,7 @@ class IColumn : public COW<IColumn> {
       LOG(FATAL) << "Method insert_many_fix_len_data is not supported for " << 
get_name();
     }
  
-    virtual void insert_many_dict_data(const int32_t* data_array, size_t 
start_index, const StringRef* dict, size_t num) {
+    virtual void insert_many_dict_data(const int32_t* __restrict data_array, 
size_t start_index, const StringRef* __restrict dict, size_t num) {

Review comment:
       I think here `__restrict` is useless, because function is null.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to