[PATCH] D48229: [NFC][SCEV] Add tests related to bit masking (PR37793)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineShifts.cpp:625-629
-    // Be careful about hiding shl instructions behind bit masks. They are used
-    // to represent multiplies by a constant, and it is important that simple
-    // arithmetic expressions are still recognizable by scalar evolution.
-    // The inexact versions are deferred to DAGCombine, so we don't hide shl
-    // behind a bit mask.
----------------
Better to just add a 'FIXME' note until we actually make the code change here to ignore 'exact'?


Repository:
  rL LLVM

https://reviews.llvm.org/D48229