areusch commented on a change in pull request #8019:
URL: https://github.com/apache/tvm/pull/8019#discussion_r632031281



##########
File path: src/tir/analysis/calculate_workspace.cc
##########
@@ -33,10 +33,12 @@ class WorkspaceCalculator : public StmtExprVisitor {
  public:
   WorkspaceCalculator() = default;
   size_t operator()(const PrimFunc& func);
+  size_t byte_alignment = 1;

Review comment:
       maybe this should become a constant somewhere just to make sure we use 
the correct default always. this isn't ideal--if Target was the correct place 
for this I'd just suggest we make it the default there. but perhaps this can be 
a stopgap til we fix Target




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to