Skip to content

Use GetFetchStackTrace for the AT_* error macros too.#13007

Closed
ezyang wants to merge 3 commits intomasterfrom
export-D10518499
Closed

Use GetFetchStackTrace for the AT_* error macros too.#13007
ezyang wants to merge 3 commits intomasterfrom
export-D10518499

Conversation

@ezyang
Copy link
Copy Markdown
Contributor

@ezyang ezyang commented Oct 23, 2018

Stack:
    :black_circle:  #13007 Use GetFetchStackTrace for the AT_ error macros too.*  💚

No reason to use the hook if it's set, this helps fbcode traces.

This slightly pessimizes the stack trace for ATen functions,
because we are no longer skipping all of the frames we should.
This is probably OK.

Differential Revision: D10518499

Differential Revision: D10518499
Differential Version: 61450239
Differential Revision: D10518499
Differential Version: 61450706
@ezyang
Copy link
Copy Markdown
Contributor Author

ezyang commented Oct 24, 2018

Hmm, I wonder why this regex is failing. Looks OK to me.

AssertionError: ".*?Don't know how to map unexpected argument (foo|bar) \(from operator .*?\).*$" does not match "[enforce fail at backend.cc:1358] . Don't know how to map unexpected argument foo (from operator Add)frame #0: c10::ThrowEnforceNotMet(char const*, int, char const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, void const*) + 0x76 (0x7fbe381706c6 in /usr/local/caffe2/lib/libc10.so)

Differential Revision: D10518499
Differential Version: 61514037
@soumith soumith deleted the export-D10518499 branch February 21, 2019 23:24
@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#13007

No reason to use the hook if it's set, this helps fbcode traces.

This slightly pessimizes the stack trace for ATen functions,
because we are no longer skipping all of the frames we should.
This is probably OK.

Reviewed By: Yangqing

Differential Revision: D10518499

fbshipit-source-id: be54e490df3c3fde7ff894b5b1473442ffc7ded3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants