forked from OSchip/llvm-project
Store bracket locations for array subscript expressions, from Erik Verbruggen!
llvm-svn: 135275
This commit is contained in:
parent
397745e540
commit
e9d62935d3
|
@ -797,9 +797,10 @@ class DeclRefExpr : public Expr {
|
|||
void computeDependence();
|
||||
|
||||
public:
|
||||
DeclRefExpr(ValueDecl *D, QualType T, ExprValueKind VK, SourceLocation L)
|
||||
DeclRefExpr(ValueDecl *D, QualType T, ExprValueKind VK, SourceLocation L,
|
||||
const DeclarationNameLoc &LocInfo = DeclarationNameLoc())
|
||||
: Expr(DeclRefExprClass, T, VK, OK_Ordinary, false, false, false, false),
|
||||
D(D), Loc(L) {
|
||||
D(D), Loc(L), DNLoc(LocInfo) {
|
||||
DeclRefExprBits.HasQualifier = 0;
|
||||
DeclRefExprBits.HasExplicitTemplateArgs = 0;
|
||||
DeclRefExprBits.HasFoundDecl = 0;
|
||||
|
|
|
@ -38,8 +38,10 @@ using namespace sema;
|
|||
/// function.
|
||||
static ExprResult
|
||||
CreateFunctionRefExpr(Sema &S, FunctionDecl *Fn,
|
||||
SourceLocation Loc = SourceLocation()) {
|
||||
ExprResult E = S.Owned(new (S.Context) DeclRefExpr(Fn, Fn->getType(), VK_LValue, Loc));
|
||||
SourceLocation Loc = SourceLocation(),
|
||||
const DeclarationNameLoc &LocInfo = DeclarationNameLoc()){
|
||||
ExprResult E = S.Owned(new (S.Context) DeclRefExpr(Fn, Fn->getType(),
|
||||
VK_LValue, Loc, LocInfo));
|
||||
E = S.DefaultFunctionArrayConversion(E.take());
|
||||
if (E.isInvalid())
|
||||
return ExprError();
|
||||
|
@ -8878,7 +8880,10 @@ Sema::CreateOverloadedArraySubscriptExpr(SourceLocation LLoc,
|
|||
ResultTy = ResultTy.getNonLValueExprType(Context);
|
||||
|
||||
// Build the actual expression node.
|
||||
ExprResult FnExpr = CreateFunctionRefExpr(*this, FnDecl, LLoc);
|
||||
DeclarationNameLoc LocInfo;
|
||||
LocInfo.CXXOperatorName.BeginOpNameLoc = LLoc.getRawEncoding();
|
||||
LocInfo.CXXOperatorName.EndOpNameLoc = RLoc.getRawEncoding();
|
||||
ExprResult FnExpr = CreateFunctionRefExpr(*this, FnDecl, LLoc, LocInfo);
|
||||
if (FnExpr.isInvalid())
|
||||
return ExprError();
|
||||
|
||||
|
|
|
@ -8357,11 +8357,24 @@ TreeTransform<Derived>::RebuildCXXOperatorCallExpr(OverloadedOperatorKind Op,
|
|||
return SemaRef.CreateOverloadedUnaryOp(OpLoc, Opc, Functions, First);
|
||||
}
|
||||
|
||||
if (Op == OO_Subscript)
|
||||
return SemaRef.CreateOverloadedArraySubscriptExpr(Callee->getLocStart(),
|
||||
OpLoc,
|
||||
First,
|
||||
Second);
|
||||
if (Op == OO_Subscript) {
|
||||
SourceLocation LBrace;
|
||||
SourceLocation RBrace;
|
||||
|
||||
if (DeclRefExpr *DRE = dyn_cast<DeclRefExpr>(Callee)) {
|
||||
DeclarationNameLoc &NameLoc = DRE->getNameInfo().getInfo();
|
||||
LBrace = SourceLocation::getFromRawEncoding(
|
||||
NameLoc.CXXOperatorName.BeginOpNameLoc);
|
||||
RBrace = SourceLocation::getFromRawEncoding(
|
||||
NameLoc.CXXOperatorName.EndOpNameLoc);
|
||||
} else {
|
||||
LBrace = Callee->getLocStart();
|
||||
RBrace = OpLoc;
|
||||
}
|
||||
|
||||
return SemaRef.CreateOverloadedArraySubscriptExpr(LBrace, RBrace,
|
||||
First, Second);
|
||||
}
|
||||
|
||||
// Create the overloaded operator invocation for binary operators.
|
||||
BinaryOperatorKind Opc = BinaryOperator::getOverloadedOpcode(Op);
|
||||
|
|
Loading…
Reference in New Issue