Work around empty set default parameter compilation issue.

Older versions of Clang don't allow using an empty set as the default parameter of a class with an explicit constructor. "error: chosen constructor is explicit in copy-initialization" BUG=chromium:630728 Change-Id: I580073788ce3346d1ecffab336a0fcee210b2e0f Reviewed-on: https://chromium-review.googlesource.com/431080Reviewed-by: Jim Stichnoth <stichnot@chromium.org>

Work around empty set default parameter compilation issue.
8bd18e1b · Nicolas Capens · Nicolas Capens · 5c4d677c · 8bd18e1b · 8bd18e1b
Commit 8bd18e1b authored Jan 20, 2017 by Nicolas Capens Committed by Nicolas Capens Jan 24, 2017
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

IceRegAlloc.h src/IceRegAlloc.h +1 -1

IceTargetLowering.cpp src/IceTargetLowering.cpp +2 -1

No files found.
--- a/src/IceRegAlloc.h
+++ b/src/IceRegAlloc.h
@@ -32,7 +32,7 @@ class LinearScan {
 public:
  explicit LinearScan(Cfg *Func);
-  void init(RegAllocKind Kind, CfgSet<Variable *> ExcludeVars = {});
+  void init(RegAllocKind Kind, CfgSet<Variable *> ExcludeVars);
  void scan(const SmallBitVector &RegMask, bool Randomized);
  // Returns the number of times some variable has been assigned a register but
  // later evicted because of a higher-priority allocation.  The idea is that we

--- a/src/IceTargetLowering.cpp
+++ b/src/IceTargetLowering.cpp
@@ -514,8 +514,9 @@ void TargetLowering::regAlloc(RegAllocKind Kind) {
    RegExclude |= RegSet_FramePointer;
  SmallBitVector RegMask = getRegisterSet(RegInclude, RegExclude);
  bool Repeat = (Kind == RAK_Global && getFlags().getRepeatRegAlloc());
+  CfgSet<Variable *> EmptySet;
  do {
-    LinearScan.init(Kind);
+    LinearScan.init(Kind, EmptySet);
    LinearScan.scan(RegMask, getFlags().getRandomizeRegisterAllocation());
    if (!LinearScan.hasEvictions())
      Repeat = false;