[PATCH] D40218: [Clang] Add __builtin_launder

[Eric Fiselier via Phabricator via cfe-commits]

EricWF added a comment.

In https://reviews.llvm.org/D40218#1298943, @Romain-Geissler-1A wrote:

> Is there any news on this code review ? Is it ready to land ?


I think so, but there were some changes regarding incomplete types since Richard last looked at it.
I wanted him to sign off on the new behavior.



================
Comment at: lib/CodeGen/CGBuiltin.cpp:1037
+
+  // FIXME: We either have an incomplete class type, or we have a class template
+  // whose instantiation has not been forced. Example:
----------------
@rsmith I believe this changed since you approved the patch.

We talked about this offline, but could you give it a thumbs up?


https://reviews.llvm.org/D40218