qinkunbao wrote: > Make Glob a vector added in parsing order
Yeah, that is needed. > Make sections a vector added in parsing order It is not necessary. > duplicate entries is not a problem as they should not be common It is not common but we need to iterate all the sections every time to ensure the correctness. ``` [sec1] src:a.txt src:b.txt [sec1] src:b.txt [sec1] src:b.txt ``` Suppose we have the function query `inSectionBlame(Section="sec1", Prefix="src", Query="a.txt", we need to iterate all sections to find the entry and get the correct line number `2`. https://github.com/llvm/llvm-project/pull/140821 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits