Bug 151381 – [JSC] Make LBasicBlock into an alias for B3::BasicBlock. #ifdef anything that does not compile

• Component: New Bugs
• Status: RESOLVED FIXED

[Benjamin Poulain]

[JSC] Make LBasicBlock into an alias for B3::BasicBlock. #ifdef anything that does not compile

[Benjamin Poulain]

Created attachment 265732 [details]
Patch

[WebKit Commit Bot]

Attachment 265732 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/ftl/FTLB3Output.h:33:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 19 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Benjamin Poulain]

Created attachment 265774 [details]
Patch

[Filip Pizlo]

Comment on attachment 265774 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=265774&action=review

> Source/JavaScriptCore/ftl/FTLAbbreviatedTypes.h:59
> +enum class LType {
> +    VoidType,
> +    Boolean,
> +    Int8,
> +    Int16,
> +    Int32,
> +    Int64,
> +    IntPtr,
> +    FloatType,
> +    DoubleType,
> +    Ref8,
> +    Ref16,
> +    Ref32,
> +    Ref64,
> +    RefPtr,
> +    RefFloat,
> +    RefDouble
> +};

Why do we need all of these types?

B3 has no notion of most of them.

> Source/JavaScriptCore/ftl/FTLCommonValues.h:74
>      const LType refPtr;
>      const LType refFloat;
>      const LType refDouble;
> -    const LValue booleanTrue;
> -    const LValue booleanFalse;
> -    const LValue int8Zero;
> -    const LValue int32Zero;
> -    const LValue int32One;
> -    const LValue int64Zero;
> -    const LValue intPtrZero;
> -    const LValue intPtrOne;
> -    const LValue intPtrTwo;
> -    const LValue intPtrThree;
> -    const LValue intPtrFour;
> -    const LValue intPtrEight;
> -    const LValue intPtrPtr;
> -    const LValue doubleZero;
> +    const LValue booleanTrue { nullptr };
> +    const LValue booleanFalse { nullptr };
> +    const LValue int8Zero { nullptr };
> +    const LValue int32Zero { nullptr };
> +    const LValue int32One { nullptr };
> +    const LValue int64Zero { nullptr };
> +    const LValue intPtrZero { nullptr };
> +    const LValue intPtrOne { nullptr };
> +    const LValue intPtrTwo { nullptr };
> +    const LValue intPtrThree { nullptr };
> +    const LValue intPtrFour { nullptr };
> +    const LValue intPtrEight { nullptr };
> +    const LValue intPtrPtr { nullptr };
> +    const LValue doubleZero { nullptr };

It might be better to say that if FTL_USES_B3, then FTLCommonValues doesn't have references to any types that B3 doesn't recognize.

[Benjamin Poulain]

Created attachment 265779 [details]
Patch

[Benjamin Poulain]

Created attachment 265782 [details]
Patch

[WebKit Commit Bot]

Attachment 265782 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp:2986:  Wrong number of spaces before statement. (expected: 24)  [whitespace/indent] [4]
ERROR: Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp:2987:  Wrong number of spaces before statement. (expected: 24)  [whitespace/indent] [4]
Total errors found: 2 in 20 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Benjamin Poulain]

Committed r192591: <http://trac.webkit.org/changeset/192591>