[PATCH] D42374: [RFC] Add IsFixed field to ISD::ArgFlagsTy

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 10:47:49 PST 2018


dsanders added inline comments.


================
Comment at: include/llvm/CodeGen/GlobalISel/CallLowering.h:53-54
+        : Reg(Reg), Ty(Ty), Flags(Flags), IsFixed(IsFixed) {
+      if (IsFixed)
+        Flags.setFixed();
+    }
----------------
It looks like SelectionDAG and FastISel are both calling Flags.setFixed() themselves. Do we still need the IsFixed argument or is GlobalISel still using it?


Repository:
  rL LLVM

https://reviews.llvm.org/D42374





More information about the llvm-commits mailing list