[PATCH] D84225: [CFE] Add nomerge function attribute to inline assembly.
Dávid Bolvanský via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 14 12:00:50 PST 2022
xbolva00 added a subscriber: aaron.ballman.
xbolva00 added inline comments.
================
Comment at: clang/lib/Sema/SemaStmtAttr.cpp:186
void VisitCallExpr(const CallExpr *E) { FoundCallExpr = true; }
+ void VisitAsmStmt(const AsmStmt *S) { FoundCallExpr = true; }
----------------
pengfei wrote:
> xbolva00 wrote:
> > This is totally wrong, just big hack to smuggle it here.
> Could you explain more? Is there any unexpect sideeffect by this?
It looks unfortunate to have something like AsmStmt in "CallExprFinder" with CallExpr as reference to clang's CallExpr.
Kinda surprised that your list of reviewers missed ALL known clang developers/code owner, in this case especially @aaron.ballman .
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84225/new/
https://reviews.llvm.org/D84225
More information about the cfe-commits
mailing list