• Seth Brenith's avatar
    [torque] Make return types required · 25f0e329
    Seth Brenith authored
    Currently, it is possible to declare macros, builtins, etc., without
    specifying a return type, in which case the return type is treated as
    void. This is confusing; the code is more clear if we require the return
    type to be specified.
    
    Aside from src/torque, this change is almost entirely just adding
    `: void` until the compiler is happy. However, two intrinsics in
    src/builtins/torque-internal.tq have been corrected to declare an
    appropriate return type. Those two intrinsics were only used in code
    generated within the compiler after the type-checking phase, so we never
    noticed that their return types were declared incorrectly.
    
    Bug: v8:7793
    Change-Id: Ib7df88678c25393a9e3eba389a6a1c4d9233dcbb
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3176502
    Commit-Queue: Seth Brenith <seth.brenith@microsoft.com>
    Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
    Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#77178}
    25f0e329
Name
Last commit
Last update
..
benchmarks Loading commit data...
bigint Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
debugging Loading commit data...
fuzzer Loading commit data...
fuzzilli Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...