[Mlir-commits] [mlir] [mlir][vector] Rename ConvertVectorToLLVM to NaiveConvertVectorToLLVM (PR #160055)

[Renato Golin]

rengolin wrote:

Agree with @dcaballe on  breaking it down, on future prs. My view is that naive is always opinionated anyway, so the name isn't too bad.

Targets usually name these simplistic lowering as C semantics (e.g. C API), which is functional but no performance promises. Usually exist for portability, which is close enough to this intent, but we have no C semantics here.

Hence, I think naive is a reasonable name. Basic, simple, direct, literal are all synonyms with the same intent, so to me, any of those names work just as well. 

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