First, that would fix the current date in the document. It would not be the date that the document was created.
Second, use the CreateDate field in a template instead. Then, you will not need a macro. The field will update each time a document is created from the template and remain fixed. You are trying to reinvent the wheel.