We are not checking if the first and second operands are masked.
patch is D60660
Paths 
  |  Differential  D60688  
[InstCombine] add failing tests of missed optimizations of fshl/fshr ClosedPublic Authored by shawnl on Apr 15 2019, 5:11 AM. 
Details 
Diff Detail 
 Event TimelineThis revision is now accepted and ready to land.Apr 16 2019, 11:57 AM Closed by commit rL358514: [InstCombine] Add tests for fshl/fshr with masked operands; NFC (authored by nikic).  ·  Explain WhyApr 16 2019, 12:08 PM This revision was automatically updated to reflect the committed changes. 
Diff 195429 llvm/trunk/test/Transforms/InstCombine/fsh.ll
  | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||