Commit 0b1c8fd
committed
[lldb][ClangExpressionParser] Remove duplicate construction of ExternalASTSourceWrapper
This is an oversight from llvm#104817 where the intention
was to hoist the ExternalASTSourceWrapper construction out of the
conditional so it can be set on both the `SemaSourceWithPriorities` and
be added as an external source to Sema. But the inner
`ExternalASTSourceWrapper` allocation wasn't actually removed.
This currently all works fine because all these AST sources are
refcounted and point to the same underlying AST sources. But this
patch cleans this up regardless.1 parent 91e09c3 commit 0b1c8fd
File tree
1 file changed
+0
-2
lines changed- lldb/source/Plugins/ExpressionParser/Clang
1 file changed
+0
-2
lines changedLines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1229 | 1229 | | |
1230 | 1230 | | |
1231 | 1231 | | |
1232 | | - | |
1233 | | - | |
1234 | 1232 | | |
1235 | 1233 | | |
1236 | 1234 | | |
| |||
0 commit comments