This revision was automatically updated to reflect the committed changes.
Closed by commit rL310472: [X86][Ms-InlineAsm] Extend MS Dot operator to accept
"this" + struct/class… (authored by coby).
Changed prior to commit:
https://reviews.llvm.org/D36450?vs=110145&id=110377#toc
Repository:
rL
coby added inline comments.
Comment at: lib/Sema/SemaStmtAsm.cpp:702-705
+ // MS InlineAsm often uses struct pointer aliases as a base
+ const QualType QT = TD->getUnderlyingType();
+ RT = isa(QT) ? QT->getPointeeType()->getAs() :
+
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Comment at: lib/Sema/SemaStmtAsm.cpp:702-705
+ // MS InlineAsm often uses struct pointer aliases as a base
+ const QualType QT = TD->getUnderlyingType();
+ RT =
coby created this revision.
Herald added a subscriber: eraman.
MS InlineAsm Dot operator accepts "Bases" such as "this" (cpp) and class/struct
pointer typedef.
This patch enhance its implementation with this behavior.
Repository:
rL LLVM
https://reviews.llvm.org/D36450
Files:
lib/Sema/Sem