rjmccall added a comment.

One minor revision, but otherwise looks great, thank you.



================
Comment at: lib/CodeGen/CodeGenModule.cpp:1166
+    if (!IsForDefinition)
+      getTargetCodeGenInfo().setTargetAttributes(FD, F, *this);
+  }
----------------
I think you should probably pass IsForDefinition to setTargetAttributes.  
Targets may want to only set certain attributes on function definitions.


Repository:
  rL LLVM

https://reviews.llvm.org/D35479



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to