[PATCH] D40330: Separate ExecutionDepsFix into 3 parts - enable breaking false dependencies for all reg classes.

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 09:26:55 PST 2017


zvi added a comment.

LGTM with a minor comment. Thanks!



================
Comment at: lib/CodeGen/ExecutionDepsFix.cpp:183
 
-  // Set up LiveRegs to represent registers entering MBB.
-  if (!LiveRegs)
-    LiveRegs = new LiveReg[NumRegs];
+  // Set up Defs to represent registers entering MBB.
+  if (!Defs.size())
----------------
This is more compact and safe:

```
Defs.assign(numRegUnits, DefaultDefVal);
```


Repository:
  rL LLVM

https://reviews.llvm.org/D40330





More information about the llvm-commits mailing list