mozga-intel opened a new pull request #20581: URL: https://github.com/apache/incubator-mxnet/pull/20581
## Description ## The size of a stack is always 4. A new optional<T> ([#658](https://github.com/dmlc/dmlc-core/pull/658)) structure is used to indicate that an object t may be "moved from" ~ allowing the efficient transfer of resources from t to another object. If an old value is moved into a new object, then the size is not enough and a blind value is reading out of the memory ~ hence: a double corruption error shows up. The size of the stack needs to be greater to avoid double corruption errors. The issue comes from #20560. ## Checklist ## ### Essentials ### - [ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc) - [ ] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage - [ ] Code is well-documented ### Changes ### - [ ] Feature1, tests, (and when applicable, API doc) - [ ] Feature2, tests, (and when applicable, API doc) ## Comments ## - If this change is a backward incompatible change, why must this change be made. - Interesting edge cases to note here -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
