This revision was automatically updated to reflect the committed changes.
Closed by commit rC347306: [ASTImporter] Set redecl chain of functions before
any other import (authored by martong, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D53702?vs=171093&id=174763#toc
Reposi
martong added a comment.
> Did you ever resolve the issue of libcxx tests not running
> https://reviews.llvm.org/D53697
Oh, by the way this change is completely independent from that other patch.
Repository:
rC Clang
https://reviews.llvm.org/D53702
___
martong added a comment.
> Did you ever resolve the issue of libcxx tests not running
> https://reviews.llvm.org/D53697
Hi @shafik ,
Sorry for the late reaction, I was on a two weeks long vacation recently.
My priority is to make those libcxx tests pass, I did not forget. :)
Repository:
rC C
shafik added a comment.
Did you ever resolve the issue of libcxx tests not running
https://reviews.llvm.org/D53697
Repository:
rC Clang
https://reviews.llvm.org/D53702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
a_sidorin accepted this revision.
a_sidorin added a comment.
This revision is now accepted and ready to land.
Herald added a reviewer: shafik.
Herald added a subscriber: gamesh411.
Hi Gabor,
The change looks fine. Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D53702
___
martong created this revision.
martong added a reviewer: a_sidorin.
Herald added subscribers: cfe-commits, Szelethus, dkrupp, rnkovacs.
Herald added a reviewer: a.sidorin.
FunctionDecl import starts with a lookup and then we create a new Decl.
Then in case of CXXConstructorDecl we further import o