[PATCH] D129142: [LLD][ELF] Drop duplicates from rpath

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

In D129142#3631738 <https://reviews.llvm.org/D129142#3631738>, @sberg wrote:

> In D129142#3631023 <https://reviews.llvm.org/D129142#3631023>, @MaskRay wrote:
>
>> Does this change the behavior in any way?
>
> Yes, the non-degenerate test case changes from
>
>   Library runpath: [/path1:/path12:/path2:/path2:/path1]
>
> to
>
>   Library runpath: [/path1:/path12:/path2]

I did not mean this behavior. I mean whether this complexity brings anything beneficial. I don't think the runtime behavior is affected.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129142/new/

https://reviews.llvm.org/D129142