[Mlir-commits] [mlir] [MLIR][Arith] Add ExpandOps to convertArithToLLVM (PR #117305)

[Hugo Trachino]

================
@@ -503,6 +505,7 @@ struct ArithToLLVMDialectInterface : public ConvertToLLVMPatternInterface {
   void populateConvertToLLVMConversionPatterns(
       ConversionTarget &target, LLVMTypeConverter &typeConverter,
       RewritePatternSet &patterns) const final {
+    arith::populateArithExpandOpsPatterns(patterns);
     arith::populateArithToLLVMConversionPatterns(typeConverter, patterns);
   }
 };
----------------
nujaa wrote:

I am not completely sure of what you mean. 
The tests `mlir/test/Conversion/ArithToLLVM/arith-to-llvm.mlir` calls both `convert-arith-to-llvm` and `--convert-to-llvm`. If either arith-to-llvm or convert-to-llvm is updated, the other must be updated as well.

https://github.com/llvm/llvm-project/pull/117305