I opened a bug for this issue: crbug/467099
Also, but all-purpose bug for /analyze is 427616 so any fix should be
annotated
with that bug also.
https://codereview.chromium.org/1000883003/diff/1/src/compiler/control-reducer.cc
File src/compiler/control-reducer.cc (right):
https://codereview.chromium.org/1000883003/diff/1/src/compiler/control-reducer.cc#newcode551
src/compiler/control-reducer.cc:551: node1->opcode() ==
IrOpcode::kIfFalse)) &&
This test looks wrong. This showed up as a new VC++ /analyze warning:
src\v8\src\objects.cc(2128) : warning C6237: (<zero> && <expression>) is
always zero. <expression> is never evaluated and might have side
effects.
I'm not sure what the correct check is -- presumably either testing for
both being kIfTrue or both being kIfFalse, or else checking for node0/1
being kIfTrue/kIfFalse or node0/1 being kIfFalse/kIfTrue. Or maybe
something else.
https://codereview.chromium.org/1000883003/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
