[libcxx-commits] [libcxx] Revert "[libc++] Avoid constructing additional objects when using map::at" (PR #160738)

[Paul Kirth via libcxx-commits]

ilovepi wrote:


> Again, I don't think this is reasonable. I'm basically working on a different code base and know almost nothing about the backend. I don't even build that part of the monorepo.
> 
> All I'm asking here is for some communication what to actually do. Why is that so hard?
> 

I think most of us assume that folks who are maintainers will be familiar w/ all this. I don't think anyone thought they weren't communicating something specific to you in the threads (I certainly thought things were clear, but obviously I misread the situation ... so sorry we didn't pick up on it earlier).

I don't think all the bots do this, but the sanitizer bots are pretty great, because they include reproducer instructions. If you look at the bottom of the log in the bot https://lab.llvm.org/buildbot/#/builders/169/builds/15293 theres a link to tell you what to do w/ a local llvm checkout https://github.com/google/sanitizers/wiki/SanitizerBotReproduceBuild

TBH its pretty awesome that its so easy. IIRC some of the old build bots always used to give you a script to run that would do something similar, which would generally be appreciated, but IIRC all bots have the CMake invocation in their logs, so it shouldn't be too hard to reproduce, generally.

Hope that helps for next time :)

https://github.com/llvm/llvm-project/pull/160738