<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 3, 2020 at 4:54 PM Paul C. Anagnostopoulos <<a href="mailto:paul@windfall.com">paul@windfall.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This just got weird. The 'const' was left over from previous experiments, so I removed it. Now when I run the -gen-instr-info backend on the AMDGPU target, I get:<br>
<br>
error: ERROR: No 'Target' subclasses defined!<br>
<br>
This suggests that there is a fundamental flaw somewhere. That message would be explained if the records pointed to by the vector of <Record *> were disappearing after the vector is stored in the map.<br></blockquote><div><br>The try_emplace solution would only work if the key mapping wasn't already present in the map. If it is already in the map, then the x[y] = z approach would be what you want.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
At 12/3/2020 07:24 PM, David Blaikie wrote:<br>
<br>
<br>
>On Thu, Dec 3, 2020 at 4:19 PM Paul C. Anagnostopoulos via llvm-dev <<mailto:<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>I have the following map:<br>
><br>
>  std::map<std::string, std::vector<Record *>> ClassRecordsMap;<br>
><br>
>I call a function that builds a new vector of <Record *> and returns it.<br>
><br>
>  const auto Records = getAllDerivedDefinitions(makeArrayRef(ClassName));<br>
><br>
><br>
>This would be one of the times where using top level const on local variables will make things less good. You can't move from Records if it's const like that. (generally I don't think it's practical to use top level const in C++ pervasively - so I'd suggest generally not doing it, because doing it only sometimes may make things harder to read ("why is /this/ particular local variable const when lots of others aren't?" the future reader will ask themselves))<br>
> <br>
><br>
>What is the best way to add it to the map, being careful not to copy it? This is what I'm doing now:<br>
><br>
>  ClassRecordsMap[ClassNameString] = std::move(Records);<br>
><br>
><br>
>Yep, generally that'll do it.<br>
><br>
>If you want to avoid even a default construction of std::vector (it's cheap, there's unlikely to be any reason to avoid that) you could use emplace:<br>
><br>
>ClassRecordsMap.try_emplace(ClassNameString, std::move(Records));<br>
<br>
</blockquote></div></div>