1221177 - Add a ByFilename breakdown for JS::ubi::Node censuses

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Comment 1

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Attachment: Add a ByFilename breakdown for JS::ubi::Node censuses

Comment 2

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4051f7875cc

Comment 3

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8682566 [details] [diff] [review]
Add a ByFilename breakdown for JS::ubi::Node censuses

Review of attachment 8682566 [details] [diff] [review]:
-----------------------------------------------------------------

seems believable

::: js/src/vm/CommonPropertyNames.h
@@ +158,5 @@
>      macro(NFKC, NFKC, "NFKC") \
>      macro(NFKD, NFKD, "NFKD") \
>      macro(nonincrementalReason, nonincrementalReason, "nonincrementalReason") \
> +    macro(noFilename, noFilename, "noFilename") \
> +    macro(noStack, noStack, "noStack")                                  \

extra spaces

::: js/src/vm/UbiNodeCensus.cpp
@@ +317,5 @@
> +    if (!obj)
> +        return nullptr;
> +
> +    for (auto entryPtr = entries.begin(); entryPtr < entries.end(); entryPtr++) {
> +        auto& entry = **entryPtr;

This can't be

  for (auto entry : entries)

somehow?

@@ +769,5 @@
> +
> +  public:
> +    ByFilename(Census& census,
> +                  CountTypePtr& thenType,
> +                  CountTypePtr& noFilenameType)

indentation

Comment 4

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

(In reply to Steve Fink [:sfink, :s:] from comment #3)
> ::: js/src/vm/UbiNodeCensus.cpp
> @@ +317,5 @@
> > +    if (!obj)
> > +        return nullptr;
> > +
> > +    for (auto entryPtr = entries.begin(); entryPtr < entries.end(); entryPtr++) {
> > +        auto& entry = **entryPtr;
> 
> This can't be
> 
>   for (auto entry : entries)
> 
> somehow?

For some reason I thought that mozilla::Vector didn't support range based for loops, but it turns out it does! \o/

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b55c75fb0c20

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b55c75fb0c20