Skip to content

Add include of cstdint to permute_vector.h#711

Merged
sunshinemyson merged 1 commit into
VeriSilicon:mainfrom
Saur2000:main
Oct 13, 2025
Merged

Add include of cstdint to permute_vector.h#711
sunshinemyson merged 1 commit into
VeriSilicon:mainfrom
Saur2000:main

Conversation

@Saur2000

Copy link
Copy Markdown
Contributor

This avoids the following error with GCC 15:

  src/tim/transform/ops/../permute_vector.h:41:11: error: 'uint32_t'
  does not name a type
     41 |   virtual uint32_t Rank() const = 0;
        |           ^~~~~~~~
  src/tim/transform/ops/../permute_vector.h:32:1: note: 'uint32_t' is
  defined in header '<cstdint>'; this is probably fixable by adding
  '#include <cstdint>'
     31 | #include <string>
    +++ |+#include <cstdint>
     32 |

@xuke537

xuke537 commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

@Saur2000 Please rebase to the last commit to fix the CI issue.

This avoids the following error with GCC 15:

  src/tim/transform/ops/../permute_vector.h:41:11: error: 'uint32_t'
  does not name a type
     41 |   virtual uint32_t Rank() const = 0;
        |           ^~~~~~~~
  src/tim/transform/ops/../permute_vector.h:32:1: note: 'uint32_t' is
  defined in header '<cstdint>'; this is probably fixable by adding
  '#include <cstdint>'
     31 | #include <string>
    +++ |+#include <cstdint>
     32 |
@Saur2000

Copy link
Copy Markdown
Contributor Author

@Saur2000 Please rebase to the last commit to fix the CI issue.

Done.

@sunshinemyson sunshinemyson merged commit 3c83eca into VeriSilicon:main Oct 13, 2025
13 checks passed
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.

3 participants