Skip to content

[ppc64le] Refer to the correct path for including ppc_vsx-inl.h in highway pkg#398

Closed
mdafsanhossain wants to merge 2 commits intoenvoyproxy:release/v1.35from
mdafsanhossain:highway-ppc
Closed

[ppc64le] Refer to the correct path for including ppc_vsx-inl.h in highway pkg#398
mdafsanhossain wants to merge 2 commits intoenvoyproxy:release/v1.35from
mdafsanhossain:highway-ppc

Conversation

@mdafsanhossain
Copy link
Copy Markdown
Contributor

Commit Message: Patch to refer to the correct path for including ppc_vsx-inl.h while building highway.
Additional Description: This fixes the following error which happens when compiler looks for the file in /usr/include while building highway as an external dependency.

ERROR: /home/user/.cache/bazel/_bazel_user/43a771d06a0b358af5ad67b4e5bcbc9d/external/highway/BUILD:156:11: Compiling hwy/targets.cc [for tool] failed: (Exit 1): gcc failed: error executing CppCompile command (from target @@highway//:hwy) /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 28 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
In file included from external/highway/hwy/targets.cc:23:
external/highway/hwy/highway.h:586:10: fatal error: hwy/ops/ppc_vsx-inl.h: No such file or directory
  586 | #include "hwy/ops/ppc_vsx-inl.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Target //source/exe:envoy-static failed to build

Testing: Checked locally.
Docs Changes: No.

Signed-off-by: Md.afsan Hossain <mdafsan.hossain@ibm.com>
Signed-off-by: Md.afsan Hossain <mdafsan.hossain@ibm.com>
@mdafsanhossain
Copy link
Copy Markdown
Contributor Author

@tedjpoole @jwendell Can you take a look at this one too?

Copy link
Copy Markdown
Member

@jwendell jwendell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please submit this change to envoy upstream first? Then the repos will sync after it lands there.

@jwendell
Copy link
Copy Markdown
Member

closing, as it was submitted upstream.

@jwendell jwendell closed this Oct 17, 2025
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