[Mlir-commits] [flang] [mlir] [mlir][tblgen] add concrete create methods (PR #147168)

[Mehdi Amini]

joker-eph wrote:

I'm OK with this direction. We need to make a call on what to do about `ImplicitLocOpBuilder.h`.

Overall I'd like to hear more about this from @jpienaar @ftynse @j2kun (maybe @Mogball and @River707 ?) : are we OK to transition from `builder.create<Op>(...)` to `Op::create(builder, ...)`  for the benefit of better IDE support?

As far as I understand this is now perfectly backward compatible with existing hand-written `build` methods, so it can be a scripted mass replacement.

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