Remove useless gtSetFlags calls
authorMike Danes <onemihaid@hotmail.com>
Wed, 15 Jun 2016 18:06:36 +0000 (21:06 +0300)
committerMike Danes <onemihaid@hotmail.com>
Wed, 15 Jun 2016 18:21:33 +0000 (21:21 +0300)
commit52d6d5fca6cf21365ba450f5b25b07b2281f65a4
treee273586d03faf6e3c8c10599b983472f26d474da
parent62586663f3ca0b5cf91dd1f417ba3ddc320ad54b
Remove useless gtSetFlags calls

For xarch `gtSetFlags` returns true and that makes `genIntToIntCast` use an `and` instruction instead of `mov`/`movzx`. This serves no purpose as no code ever calls `gtRequestSetFlags` on cast nodes nor does `GT_JTRUE` expects a cast node as its operand.

`gtSetFlags` seems to be a leftover from the legacy codegen.

Commit migrated from https://github.com/dotnet/coreclr/commit/845d34814cbe4f0f8cd342d04bd47fc7ef1c79b2
src/coreclr/src/jit/codegenxarch.cpp