Issue metadata
Sign in to add a comment
|
0% regression in sizes at 618755:618756 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Yesterday
(38 hours ago)
,
Today
(21 hours ago)
I doubt given the range and the lag to the commit we can do anything. feel free to close.
,
Today
(14 hours ago)
+clemensh I have a vague suspicion that it might be due to disabling noexcept on msvc in https://chromium-review.googlesource.com/c/v8/v8/+/1386869.
,
Today
(14 hours ago)
There is no similar regression in d8 on my linux workstation.
,
Today
(13 hours ago)
We should try to bisect to a single CL.
If it's really the noexcept annotation, we can do something about it. We would basically have to wrap each standard container and mark the constructors and assignment operators noexcept. That's doable with a single template. It would be a bit ugly because we have to identify all locations where it's actually needed. Some fields would be declared as:
base::NoExcept<std::vector<Foo>>
On non-msvc, that would be a nop-typedef:
namespace base {
template<typename T>
using NoExcept = T;
}
But as I said, before doing this, we should bisect the size increase.
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Jan 2