This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG6dc9db524492: Track expression language from one place in
ClangExpressionParser (authored by EwanCrawford).
Herald added a
ldrumm added a subscriber: LLDB.
ldrumm removed rL LLVM as the repository for this revision.
ldrumm updated this revision to Diff 82251.
ldrumm added a comment.
This patch was committed back in March https://reviews.llvm.org/rL263099 and
then reverted 2 hours later in https://reviews.llvm.org/rL2
ldrumm reopened this revision.
ldrumm added a comment.
This patch was committed back in March https://reviews.llvm.org/rL263099 and
then reverted 2 hours later in https://reviews.llvm.org/rL263107 due to a
testing failure on greendragon related to ObjC. Unfortunately, I no longer have
access t
This revision was automatically updated to reflect the committed changes.
Closed by commit rL263099: Track expression language from one place in
ClangExpressionParser (authored by EwanCrawford).
Changed prior to commit:
http://reviews.llvm.org/D17719?vs=49531&id=50253#toc
Repository:
rL LLVM
ldrumm added a comment.
Hi all
I'm still keen to have someone more familiar with this code to let me know if
anything's amiss, but I'll probably move ahead and commit this tomorrow if I
don't hear anything.
Let me know if you'd like any changes made - or the patch is simply rotten to
the core
ldrumm added a comment.
Hi all
If there are no objections, I'd like to commit this. Is this in an acceptable
state?
Best
Luke
http://reviews.llvm.org/D17719
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
ldrumm updated this revision to Diff 49531.
ldrumm added a comment.
Updated diff to show full context
http://reviews.llvm.org/D17719
Files:
source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp
source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h
Index: source/Plugins/Ex
clayborg resigned from this revision.
clayborg removed a reviewer: clayborg.
clayborg added a comment.
I will let Sean review this one as the expression parser is his area of
expertise.
http://reviews.llvm.org/D17719
___
lldb-commits mailing list
l
ldrumm created this revision.
ldrumm added reviewers: clayborg, spyffe.
ldrumm added a subscriber: lldb-commits.
The current expression language is currently tracked in a few places within the
`ClangExpressionParser` constructor. This patch adds a private
`lldb::LanguageType` attribute to the `
