[PATCH] D38596: Implement attribute target multiversioning

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 8 07:28:44 PDT 2017


aaron.ballman added a comment.

The attribute and sema bits look good to me, but I agree that you might want Richard's opinions before committing.



================
Comment at: lib/Sema/SemaDecl.cpp:9264
+
+  if (auto *CMD = dyn_cast<CXXMethodDecl>(FD))
+    if (CMD->isVirtual()) {
----------------
`const auto *`


https://reviews.llvm.org/D38596





More information about the cfe-commits mailing list