lhutton1 commented on pull request #6430:
URL: https://github.com/apache/incubator-tvm/pull/6430#issuecomment-690476905


   Thanks for the pointer @anijain2305. I do agree this is a better approach. I 
have an example working on my end for my use-case, although it seems quite 
messy. Setting temporary attributes i.e. `FTVMConvertOpLayout` on the C++ side 
of things is more difficult than from python. The reason I need to do this in 
C++ is because I run the convert layout pass in Arm Compute Library codegen.
    
   I've had a go at implementing something like the `TempOpAttr` class from 
python in C++. This is to ensure I'm only setting this config for the Arm 
Compute Library codegen. Although, this involves fetching a series of packed 
functions (namely OpGetAttr, OpResetAttr and OpSetAttr) from `src/ir/op.cc` 
which doesn't sound right. I'm just wondering if it sounds like I'm completely 
off piste with this or whether you know of anything that could help that I've 
missed?


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