diff options
author | Alon Zakai <azakai@google.com> | 2023-05-10 14:05:06 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-10 14:05:06 -0700 |
commit | 7d5d24f400019ff023b65ccdb3c1d8d07ebb00a5 (patch) | |
tree | 8c3866590088c51c0ee849a36217d9a309383367 /src/wasm.h | |
parent | b1dba91a6bf881b5bfa68da7bb56bbb3bbf90a0a (diff) | |
download | binaryen-7d5d24f400019ff023b65ccdb3c1d8d07ebb00a5.tar.gz binaryen-7d5d24f400019ff023b65ccdb3c1d8d07ebb00a5.tar.bz2 binaryen-7d5d24f400019ff023b65ccdb3c1d8d07ebb00a5.zip |
Extend drop.h and use it in Directize (#5713)
This adds an option to ignore effects in the parent in
getDroppedChildrenAndAppend. With that, this becomes usable in more
places, like Directize, basically in situations where we know we can ignore
effects in the parent (since we've inferred they are not needed). This lets
us get rid of some boilerplate code in Directize.
Diff without whitespace is a lot smaller. A large other part of the diff is a
rename of curr => parent which I think it makes it more readable as then
parent/children is a clear contrast, and then the new parameter "ignore/
notice parent effects" is obviously connected to "parent".
The top comment in drop.cpp is removed as it just duplicated the top
comment in the header drop.h.
This is basically NFC but using drop.h does bring the advantage of
emitting less code, see the test changes, so it is noticeable in the IR.
This is a refactoring PR in preparation for a larger improvement to
Directize that will also benefit from this new drop capability.
Diffstat (limited to 'src/wasm.h')
0 files changed, 0 insertions, 0 deletions