mwasplund abandoned this revision.
mwasplund added a comment.
I completely forgot this was open. I have been making incremental improvements
to modules ts and this has gotten pulled into that change.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D52973/new/
https
mwasplund added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:1467-1470
+ // FIXME: The Modules TS does not specify how to handle inplicit types
+ // For now we will simply ignore the implicit global types
+ if (Old->isImplicit())
+return false;
rsmit
mwasplund added a comment.
Can you see the full change? I am only seeing that latest commit after running:
"git show HEAD -U99 > mypatch.patch".
It was working fine with I ran "git diff master..mybranch > mybranch.diff"
however that does not give the full context.
Repository:
rC Clang
mwasplund added inline comments.
Comment at: lib/Sema/SemaDecl.cpp:1464
if (NewM == OldM)
return false;
rsmith wrote:
> Somewhere up here we're calling `getOwningModule()` but should be calling
> `getOwningModuleForLinkage()`. (Please upload patches with
mwasplund updated this revision to Diff 169879.
mwasplund marked 2 inline comments as done.
Repository:
rC Clang
https://reviews.llvm.org/D52973
Files:
include/clang/AST/ASTContext.h
lib/AST/ASTContext.cpp
lib/Sema/SemaDecl.cpp
test/Modules/msvc-compat-implitic-types.cpp
Index: test/
mwasplund added a comment.
In https://reviews.llvm.org/D52973#1259909, @rsmith wrote:
> Generally this looks good, but it needs an accompanying test.
I was looking into that and trying to read up on the documentation for adding
tests. I think I understand the crazy comment RUN test invocation,
mwasplund updated this revision to Diff 168607.
Repository:
rC Clang
https://reviews.llvm.org/D52973
Files:
include/clang/AST/ASTContext.h
include/clang/Serialization/ASTBitCodes.h
lib/AST/ASTContext.cpp
lib/Sema/Sema.cpp
lib/Sema/SemaDecl.cpp
lib/Serialization/ASTReader.cpp
lib/
mwasplund updated this revision to Diff 168606.
Repository:
rC Clang
https://reviews.llvm.org/D52973
Files:
lib/Sema/SemaDecl.cpp
Index: lib/Sema/SemaDecl.cpp
===
--- lib/Sema/SemaDecl.cpp
+++ lib/Sema/SemaDecl.cpp
@@ -5235,7
mwasplund created this revision.
Herald added a subscriber: cfe-commits.
Bug 39052 - [Modules TS] MSVC std library produces ambiguous type_info
reference when including module with ms-compatibility
When compiling a modules-ts project with MSVC compatibility turned on the
type_info class was get
