[REBASE&FF] .azurepipelines: Add CLANGPDB CI#896
Conversation
❌ QEMU Validation FailedSource Dependencies
Results
Workflow run: https://github.com/microsoft/mu_plus/actions/runs/27702808583 This comment was automatically generated by the Mu QEMU PR Validation workflow. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release/202511 #896 +/- ##
=================================================
Coverage ? 10.23%
=================================================
Files ? 149
Lines ? 24522
Branches ? 2346
=================================================
Hits ? 2509
Misses ? 21977
Partials ? 36
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
9a0c0fb to
d3f5cf2
Compare
|
@Javagedes, is the PR ready for review now? |
Yes |
os-d
left a comment
There was a problem hiding this comment.
This doesn’t actually fix the reported error; did that not repro or are we not building the lib in CI?
|
@Javagedes, this commit message has a typo that makes it difficult to read:
If you could, please fix that before merging. |
Any io the error handling conditions that cause a goto `Done` will call `if (Blt != NULL)` even though Blt may not have been initialized. This commit sets Blt to null at the beggining of the function to resolve this.
This commit adds the CONST qualifier to the CHAR8 strings to prevent memcpy from being used to copy the strings around.
Specify CLANGPDB build flags so that GCC build flags are not applied to the CLANGPDB build
d3f5cf2 to
71c46c7
Compare
Since @Javagedes is unavailable at the moment, I made that update. Given the pipeline files look reasonable and it would be beneficial to start getting coverage, I am merging the PR now. |
Description
Add CI that specifically builds each package in DEBUG and RELEASE with CLANGPDB on both Windows and Linux. Applies necessary changes to compile all packages with CLANGPDB.
How This Was Tested
Windows: https://dev.azure.com/projectmu/mu/_build/results?buildId=111589&view=results
Linux: https://dev.azure.com/projectmu/mu/_build/results?buildId=111590&view=results
Integration Instructions
N/A