================ @@ -1138,8 +1138,9 @@ mlir::Value ScalarExprEmitter::emitShl(const BinOpInfo &ops) { mlir::isa<cir::IntType>(ops.lhs.getType())) cgf.cgm.errorNYI("sanitizers"); - cgf.cgm.errorNYI("shift ops"); - return {}; + return builder.create<cir::ShiftOp>(cgf.getLoc(ops.loc), ---------------- andykaylor wrote:
Is there a reason this doesn't get a `createShift` call in `CIRBaseBuilderTy`? In particular, the `cgf.getBuilder().getUnitAttr()` argument (or lack thereof) is not very clear. A call to something like `createShiftLeft` or `createShiftRight` would be better. https://github.com/llvm/llvm-project/pull/133405 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits