[llvm-branch-commits] [DirectX] Lower `@llvm.dx.typedBufferStore` to DXIL ops (PR #104253)

[Farzon Lotfi via llvm-branch-commits]

================
@@ -289,6 +289,43 @@ class OpLowerer {
     });
   }
 
+  void lowerTypedBufferStore(Function &F) {
+    IRBuilder<> &IRB = OpBuilder.getIRB();
+    Type *Int8Ty = IRB.getInt8Ty();
+    Type *Int32Ty = IRB.getInt32Ty();
+
+    replaceFunction(F, [&](CallInst *CI) -> Error {
----------------
farzonl wrote:

The thing that's throwing me here is that we have a bunch of `replaceFunction` Lambda's and we have a replaceFunction method in `OpLowerer` and what I'm wondering is why do we do methods sometimes and lambdas others times.  further can `lowerTypedBufferLoad` share anything with `lowerTypedBufferStore` `replaceFunction ` if they weren't lambdas?

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