Fabian Ruffy

The control plane pulls in a bunch of dependencies (Protobuf and soon also Abseil). Not every back end needs these dependencies. Add a CMake option to enable/disable building the control-plane....

Add support for a clang-tidy linter to CMake. Add a files utility function to P4CUtils.cmake.

4

Add the ability to run clang-tidy across P4C files via CMake. This is not enabled yet for CI because clang-tidy is quite slow still. However, one can still use it...

Replace boost constructs with their C++17 STL equivalents

16

Inspired by https://github.com/ethereum/solidity/issues/7259. Now that we have C++17 support there is some boost code we can remove. The less boost we have, the better. For now, this can apply: -...

Now that we have increased the minimum Protobuf version we may actually be able to request a Protobuf version that includes the fixes to the CMake config file. This way...

Compiler Bug: duplicate primitive converter for modify_field at priority

6

My downstream projects are hitting the following error after recent changes: ``` terminate called after throwing an instance of 'Util::CompilerBug' what(): In file: /home/runner/work/flay/p4c/frontends/p4/fromv1.0/converters.cpp:602 Compiler Bug: duplicate primitive converter for...

RenameMap in inlining.cpp does not preserve the original variable name.

2

The original variable name is lost, leading to incorrect debugging and printing output.

It should be possible to link P4Testgen or P4Tools in general as a library. Currently, we can only call it as a binary. For that to be possible several things...

Use libbpf instead of iproute2 to load eBPF programs

2

The dependency on iproute2 and tc causes some issues when loading eBPF programs. Different versions of Ubuntu package different versions of iproute2, which offers different levels of support for eBPF....

Context in https://github.com/p4lang/p4c/pull/4377#discussion_r1483274555_

Simplify bazel cache key.

4

The Bazel CI build quickly fills up the available cache because it creates an entry for any single commit. However, the cache is loaded from the latest entry. Maybe we...