Your observations are correct - AtlasRegionAttacher indeed calls originalAttachment.GetRemappedClone which creates a new Material instance.

Thanks for reporting!

I have created a ticket here, you can subscribe to receive notifications on updates:
[unity] AtlasRegionAttacher creates unnecessary Material clones · #1285