[PATCH] D46441: [clang][CodeGenCXX] Noalias attr for copy/move constructor arguments

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 4 17:36:57 PDT 2018


efriedma added inline comments.


================
Comment at: lib/CodeGen/CodeGenModule.cpp:2512
+  if (D && isa<CXXConstructorDecl>(D) &&
+      cast<CXXConstructorDecl>(D)->isCopyOrMoveConstructor())
+    F->addParamAttr(1, llvm::Attribute::NoAlias);
----------------
Why does it matter whether it's a copy constructor?  The standard text you're quoting doesn't seem to make that distinction.


Repository:
  rC Clang

https://reviews.llvm.org/D46441





More information about the cfe-commits mailing list