Skip to content

Comments

Fix missing include#220

Closed
ArchangeGabriel wants to merge 1 commit intointel:masterfrom
ArchangeGabriel:patch-2
Closed

Fix missing include#220
ArchangeGabriel wants to merge 1 commit intointel:masterfrom
ArchangeGabriel:patch-2

Conversation

@ArchangeGabriel
Copy link
Contributor

Without this:

/build/intel-graphics-compiler/src/intel-graphics-compiler-igc-1.0.9441/visa/LocalScheduler/SWSB_G4IR.cpp: In member function ‘vISA::SBNode* vISA::SWSB::reuseTokenSelection(const vISA::SBNode*) const’:
/build/intel-graphics-compiler/src/intel-graphics-compiler-igc-1.0.9441/visa/LocalScheduler/SWSB_G4IR.cpp:1486:34: error: ‘numeric_limits’ is not a member of ‘std’
 1486 |         int maxTokenDelay = std::numeric_limits<int>::min(); //The delay may cause if reuse
      |                                  ^~~~~~~~~~~~~~
/build/intel-graphics-compiler/src/intel-graphics-compiler-igc-1.0.9441/visa/LocalScheduler/SWSB_G4IR.cpp:1486:49: error: expected primary-expression before ‘int’
 1486 |         int maxTokenDelay = std::numeric_limits<int>::min(); //The delay may cause if reuse
      |                                                 ^~~
/build/intel-graphics-compiler/src/intel-graphics-compiler-igc-1.0.9441/visa/LocalScheduler/SWSB_G4IR.cpp:1487:37: error: ‘numeric_limits’ is not a member of ‘std’
 1487 |         int minTokenDistance = std::numeric_limits<int>::max(); //The distance from the reused node
      |                                     ^~~~~~~~~~~~~~
/build/intel-graphics-compiler/src/intel-graphics-compiler-igc-1.0.9441/visa/LocalScheduler/SWSB_G4IR.cpp:1487:52: error: expected primary-expression before ‘int’
 1487 |         int minTokenDistance = std::numeric_limits<int>::max(); //The distance from the reused node
      |                                                    ^~~

@pszymich
Copy link
Contributor

Hi @ArchangeGabriel,

Your change has been merged and you can see it here: 4f51046

Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants