[PATCH] D20769: [IPRA] Interprocedural Register Allocation - Analysis Passes

Vivek Pandya via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 6 22:31:25 PDT 2016


vivekvpandya added inline comments.

================
Comment at: lib/CodeGen/RegisterUsageInfo.cpp:56
@@ +55,3 @@
+void PhysicalRegisterUsageInfo::storeUpdateRegUsageInfo(
+    const GlobalVariable* MFGlobalVar, std::vector<uint32_t> RegMask) {
+  RegMasks[MFGlobalVar] = std::move(RegMask);
----------------
mehdi_amini wrote:
> Add an assertion that `MFGlobalVar` is not null (and the name isn't very explicit)
now I think such assertion wold be for Function *, right ? 


http://reviews.llvm.org/D20769





More information about the llvm-commits mailing list