Also replace dyn_cast_or_null with dyn_cast when possible.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
A couple of nits
mlir/lib/Conversion/StandardToLLVM/ConvertStandardToLLVM.cpp | ||
---|---|---|
1169 | Is it really worth it to add a template (and hence increase compile time and binary size) only to forward it to SmallVector? I'd just use 4 instead, this should cover the vast majority of cases. | |
1170 | Please use LogicalResult instead of bool | |
1171 | Can we use ValueRange instead of ArrayRef<Value>, this is more flexible and ValueRange is implicitly constructible from ArrayRef and other things. | |
1189 | Nit: drop trivial braces |
Addressed review comments.
mlir/lib/Conversion/StandardToLLVM/ConvertStandardToLLVM.cpp | ||
---|---|---|
1169 | Actually I first did exactly that but then thought maybe I shouldn't introduce an actual change with this CL. But I completely agree with you, so changed it to SmallVector with constant 4. |
Is it really worth it to add a template (and hence increase compile time and binary size) only to forward it to SmallVector? I'd just use 4 instead, this should cover the vast majority of cases.