[llvm] [SandboxIR] Adding isVolatile arg to existing LoadInst::create() functions (PR #100781)

Julius Alexandre via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 26 13:34:00 PDT 2024


================
@@ -743,6 +743,9 @@ class LoadInst final : public Instruction {
   static LoadInst *create(Type *Ty, Value *Ptr, MaybeAlign Align,
                           Instruction *InsertBefore, Context &Ctx,
                           const Twine &Name = "");
+  static LoadInst *create(Type *Ty, Value *Ptr, MaybeAlign Align,
+                          Instruction *InsertBefore, Context &Ctx,
+                          const Twine &Name = "", bool isVolatile);
----------------
medievalghoul wrote:

It's a pain to deal with, I'll stick with the former suggestion. After trying to deal with it, I realized it creates ambiguity in the code

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


More information about the llvm-commits mailing list