[clang] [llvm] Compute GUIDs once and store in metadata (PR #184065)
Teresa Johnson via cfe-commits
cfe-commits at lists.llvm.org
Fri May 1 12:17:02 PDT 2026
================
@@ -527,14 +528,21 @@ bool llvm::runPassPipeline(
case OK_NoOutput:
break; // No output pass needed.
case OK_OutputAssembly:
+ if (EmitSummaryIndex) {
----------------
teresajohnson wrote:
Can you add a comment that this is for pre-existing tests that don't have the GUID metadata? To avoid duplication you could also pull all 3 calls out of the switch with one comment. For simplicity I don't even think you need to make it conditional, as this is for tests. But if you do make it conditional here or pulled up, no braces for single line if bodies.
https://github.com/llvm/llvm-project/pull/184065
More information about the cfe-commits
mailing list