[llvm] [SandboxIR] Added isVolatile args to existing LoadInst::create function (PR #100850)

Julius Alexandre via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 28 09:44:43 PDT 2024


================
@@ -764,11 +764,21 @@ define void @foo(ptr %arg0, ptr %arg1) {
   // Check create(InsertBefore)
   sandboxir::LoadInst *NewLd =
       sandboxir::LoadInst::create(Ld->getType(), Arg1, Align(8),
-                                  /*InsertBefore=*/Ret, Ctx, "NewLd");
+                                  /*InsertBefore=*/Ret, Ctx,
+                                  /*IsVolatile=*/false, "NewLd");
----------------
medievalghoul wrote:

Completely understandable in terms of LoadInst looking verbose. Btw, do you still want the `bool IsVolatile` to be before the `Context &Ctx` in the follow-up PR?

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


More information about the llvm-commits mailing list