I am not sure why it would be sucking up memory, but I see a couple of things that could cause long execution times.
Are you using 32-bit or 64-bit Office?
Could you provide an Excel file with a couple of dozen lines of sample data (not private data) so we can analyze the code under execution?
Your code seems reasonably straightforward although there are a couple of things I would do differently.
There is no reason to put an arbitrary number in there. I would use
It's not clear what you are trying to do here. Your comment doesn't match the code. The code iterates from columns A through ALL. Iterating
Might be better to iterate through the columns that are actually used.
Bookmarks