[PATCH] D112663: [clang-repl] Allow Interpreter::getSymbolAddress to take a mangled name.

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 30 09:01:41 PST 2021

aaron.ballman added inline comments.

Comment at: clang/unittests/Interpreter/InterpreterTest.cpp:205-207
+  // FIXME: We cannot yet handle delayed template parsing. If we run with
+  // -fdelayed-template-parsing we try adding the newly created decl to the
+  // active PTU which causes an assert.
aaron.ballman wrote:
> FWIW, Intel's internal testing is hitting crashes with this test. With `-fno-delayed-template-parsing` enabled, we get a failed assertion:
> ```
> Assertion failed: OffsetBytes <= AllocationSize && "Offset out of bounds!", file d:\iusers\ayrivera\dev-xmain-web\llvm\l
> lvm\lib\executionengine\runtimedyld\RuntimeDyldImpl.h, line 90
> ```
> and when we enable the delayed template parsing, we get a different assertion:
> ```
> Assertion failed: DC->getLexicalParent() == CurContext && "The next DeclContext should be lexically contained in the current one.", file D:/iusers/ayrivera/dev-xmain-web/llvm/clang/lib/Sema/SemaDecl.cpp, line 1305
> ```
> I think this FIXME either needs to be addressed or the patch reverted until this is addressed.
Ignore this -- this comment was saved in my Phab drafts from an earlier investigation and I managed to not delete it when sending my other comment (below). Oops!



More information about the cfe-commits mailing list