[PATCH] D27584: AMDGPU: Improve nsw/nuw/exact when promoting uniform i16 ops

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUCodeGenPrepare.cpp:221
   }
-  ExtRes = copyFlags(I, Builder.CreateBinOp(I.getOpcode(), ExtOp0, ExtOp1));
   TruncRes = Builder.CreateTrunc(ExtRes, I.getType());
----------------
kzhuravl wrote:
> Should `copyFlags` be completely removed?
Yes, I noticed this after I posted


https://reviews.llvm.org/D27584