[lld] r281754 - Recommit r281721 "[ELF] - Linkerscript: implement EXCLUDE_FILE in the middle of a input section description."

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 19 06:14:28 PDT 2016


>Can you add a comment saying that list is used to avoid a compiler (libc++?) bug? std::vector is far more common, so it is nice to justify why a list is used.

>
>
>Thanks,
>Rafael


?Rui already changed that place in r281803, to use struct instead of pair,

but seems also had problems with vector (see r281805), as comment now says:


  // We use std::list instead of std::vector because SectionPattern
  // do not support move assignment.

I am not sure it is correct comment though, as I think struct without anything like desctructors,
and just that has 2 movable fields should be movable.

There is no link on buildbot in commit message for 281805,
I`ll check if swiching to struct helped my FreeBSD VM where It was reproducible.

??

George.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160919/8c97ec1e/attachment.html>


More information about the llvm-commits mailing list