bcahoon added inline comments.
================ Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:4700 + B.getMF().getFunction().getContext().diagnose(BadIntrin); + B.buildUndef(MI.getOperand(0)); + MI.eraseFromParent(); ---------------- rampitec wrote: > rampitec wrote: > > Just return false like in other places. > Just return false. I see that is like this in the whole file. Changed this to false, and also changed SIISelLlowering to return SDValue so that both fail in a similar way. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103663/new/ https://reviews.llvm.org/D103663 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
