Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Гапон А.К. 381806-3 #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Andrey0948
Copy link

No description provided.

@eshulankina
Copy link
Owner

The program does not compile. Please fix errors.

@Andrey0948
Copy link
Author

@eshulankina поправил

Result.Context->getLangOpts());
auto res = ("static_cast<" + tSource + ">(").str();
rwr.ReplaceText(loc, res);
auto closePL = Lexer::getLocForEndOfToken(
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please describe why you are using Lexer::getLocForEndOfToken(expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc(),...) here.
What would be the difference if expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc().getLocOffset(1) was used as closePL?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lexer::getLocForEndOfToken определяет локацию за последним элементом токена

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the difference in the use with expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc().getLocOffset(1)?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну эта функция возвращает изначальное местоположение, сразу за концом токена, на который ссылается переданное местоположение expr-> getSubExprAsWritten () -> IgnoreImpCasts () -> getEndLoc ()

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happens if we use expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc().getLocOffset(1) instead of Lexer::getLocForEndOfToken()?
When does the approach with expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc().getLocOffset(1) not work?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SubExpr->getEndLoc().getLocWithOffset(1) захватил бы только первый символ имени, в отличие от реализованного варианта.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auto res = ("static_cast<" + tSource + ">(").str();
rwr.ReplaceText(loc, res);
auto closePL = Lexer::getLocForEndOfToken(
expr->getSubExprAsWritten()->IgnoreImpCasts()->getEndLoc(), 0,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please describe what getSubExprAsWritten() does.
What is the difference with getSubExpr() method?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Получает подвыражение приведения как написано в исходном коде, просматривая неявные приведения или другие промежуточные узлы, которые были добавлены семантическим анализом.
getSubExpr возвращает итоговый результат, с учетом неявных приведений и промежуточных узлов

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you try to print the AST of the test.cpp example, you will see that you have several ImplicitCastExpr nodes:
image

Depending on which approaches you use to get the subexpression node, you will get different result nodes for the test.cpp example:

Code:

// getSubExpr() retrieves the first cast subexpression
llvm::outs() << "getSubExpr(): " << castNode->getSubExpr()->getStmtClassName();

// getSubExprAsWritten() retrieves the cast subexpression as it was written in the source
// code, looking through any implicit casts or other intermediate nodes
// introduced by semantic analysis.
llvm::outs() << "getSubExprAsWritten(): " << castNode->getSubExprAsWritten()->getStmtClassName();

// getSubExpr() retrieves the first cast subexpression.
// ignoreImpCasts() skips past any implicit casts which might surround this expression until
// reaching a fixed point. It skips the following AST nodes: ImplicitCastExpr, FullExpr.
llvm::outs() << "getSubExpr()->ignoreImpCasts(): " << castNode->getSubExpr()->IgnoreImpCasts()->getStmtClassName();

Output:

 getSubExpr(): ImplicitCastExpr 
 getSubExprAsWritten(): DeclRefExpr  
 getSubExpr()->ignoreImpCasts(): DeclRefExpr

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