Skip to content

[Ubuntu24.04] llvm-project in vpux_plugin fails to compile#35

Merged
Maxim-Doronin merged 3 commits intoopenvinotoolkit:developfrom
DariaMityagina:icv/dm/fix-ubuntu-24
Jun 21, 2024
Merged

[Ubuntu24.04] llvm-project in vpux_plugin fails to compile#35
Maxim-Doronin merged 3 commits intoopenvinotoolkit:developfrom
DariaMityagina:icv/dm/fix-ubuntu-24

Conversation

@DariaMityagina
Copy link
Copy Markdown
Contributor

Summary

[Ubuntu24.04] llvm-project in vpux_plugin fails to compile due to:

/build/third_party/vpux_plugin/src/vpux_plugin/src/vpux_compiler/src/dialect/const/utils/constant_folding_cache.cpp:26:22:
/usr/include/c++/13/bits/atomic_base.h:481:25: error: 'void __atomic_store_8(volatile void*, long unsigned int, int)' writing 8 bytes into a region of size 0 overflows the destination [-Werror=stringop-overflow=]
  481 |         __atomic_store_n(&_M_i, __i, int(__m));
      |         ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~
In member function 'vpux::Const::FoldingRequest vpux::Const::ConstantFoldingCache::getRequest()':
cc1plus: note: destination object is likely at address zero
error: redundant move in initialization [-Werror=redundant-move]
      update_concat_out_fq(std::move(node), fqs_to_align);
vpux_plugin/thirdparty/llvm-project/llvm/include/llvm/ADT/FunctionExtras.h:307:7: error: '<unnamed>.llvm::unique_function<__attribute__((const)) bool(mlir::OpOperand&)>::<unnamed>.llvm::detail::UniqueFunctionBase<bool, mlir::OpOperand&>::StorageUnion.llvm::detail::UniqueFunctionBase<bool, mlir::OpOperand&>::StorageUnionT::OutOfLineStorage' may be used uninitialized [-Werror=maybe-uninitialized]
      StorageUnion.OutOfLineStorage = RHS.StorageUnion.OutOfLineStorage;
c++/13/bits/stl_algobase.h:437:30: error: 'void* __builtin_memmove(void*, const void*, long unsigned int)' forming offset 200 is out of the bounds [0, 200] of object '<anonymous>' with type 'llvm::SmallVector<VPUNN::DPULayer>' [-Werror=array-bounds=]
       __builtin_memmove(__result, __first, sizeof(_Tp) * _Num);

etc

Target Platform For Release Notes (Mandatory)

Aids the generation of release notes

  • VPUX30XX
  • VPUX31XX
  • VPUX37XX
  • NONE (Not included in release notes)

Classification of this Pull Request

  • Maintenance
  • BUG
  • Feature

Related PRs

Code Review Survey (Copy and Complete in your code review)

  • number_minutes_spent_on_review[0]
  • number_p1_defects_found[0]
  • number_p2_defects_found[0]
  • number_p3_defects_found[0]
  • number_p4_defects_found[0]

@DariaMityagina DariaMityagina added the bug Something isn't working label Jun 20, 2024
@DariaMityagina DariaMityagina self-assigned this Jun 20, 2024
@DariaMityagina DariaMityagina requested a review from a team as a code owner June 20, 2024 18:47
@DariaMityagina
Copy link
Copy Markdown
Contributor Author

Must be merged after intel/npu-plugin-llvm#44.

@DariaMityagina
Copy link
Copy Markdown
Contributor Author

@Maxim-Doronin could you please take a look?

@Maxim-Doronin Maxim-Doronin merged commit 260a105 into openvinotoolkit:develop Jun 21, 2024
Maxim-Doronin added a commit to Maxim-Doronin/npu_compiler that referenced this pull request Aug 18, 2025
Maxim-Doronin added a commit that referenced this pull request Aug 21, 2025
* UD2025.32 Update file batch #1

* UD2025.32 Update file batch #2

* UD2025.32 Update file batch #3

* UD2025.32 Update file batch #4

* UD2025.32 Update file batch #5

* UD2025.32 Update file batch #6

* UD2025.32 Update file batch #7

* UD2025.32 Update file batch #8

* UD2025.32 Update file batch #9

* UD2025.32 Update file batch #10

* UD2025.32 Update file batch #11

* UD2025.32 Update file batch #12

* UD2025.32 Update file batch #13

* UD2025.32 Update file batch #14

* UD2025.32 Update file batch #15

* UD2025.32 Update file batch #16

* UD2025.32 Update file batch #17

* UD2025.32 Update file batch #18

* UD2025.32 Update file batch #19

* UD2025.32 Update file batch #20

* UD2025.32 Update file batch #21

* UD2025.32 Update file batch #22

* UD2025.32 Update file batch #23

* UD2025.32 Update file batch #24

* UD2025.32 Update file batch #25

* UD2025.32 Update file batch #26

* UD2025.32 Update file batch #27

* UD2025.32 Update file batch #28

* UD2025.32 Update file batch #29

* UD2025.32 Update file batch #30

* UD2025.32 Update file batch #31

* UD2025.32 Update file batch #32

* UD2025.32 Update file batch #33

* UD2025.32 Update file batch #34

* UD2025.32 Update file batch #35

* UD2025.32 Update file batch #36

* UD2025.32 Update file batch #37

* UD2025.32 Update file batch #38

* UD2025.32 Update file batch #39

* UD2025.32 Update file batch #40

* UD2025.32 Update ELF

* WA for GCC warning

* Revert driver update

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants