areusch commented on pull request #6319: URL: https://github.com/apache/incubator-tvm/pull/6319#issuecomment-678362316
i guess the root problem here is we aren't able to use a higher-level language construct to describe the memory layout since we are serializing/deserializing. because of that, we're casting and can't rely on `-Werror` (my view is that most of these should be caught using `-Werror`) to really catch an underlying data format discrepancy; though, it should catch this bug. it looks like we aren't using -Werror with apps/bundle_deploy in master right now, we should probably turn that on and clean up all the warnings in a future PR. ---------------------------------------------------------------- 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]
