rnk added a comment.

In https://reviews.llvm.org/D50877#1204609, @thakis wrote:

> Can you explicitly mention that this intentionally doesn't use an absolute 
> path in MicrosoftMangleContextImpl() or similar?


Sure. I also described the issue with codeview that motivates why we want 
unique names.

I think I sent this out hastily at the end of the day yesterday, and I didn't 
write enough comments to answer all of your questions. Hopefully with these 
changes it makes more sense.


https://reviews.llvm.org/D50877



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

Reply via email to