[llvm] [CFI][LowerTypeTests] Fix indirect call with alias (PR #106185)
Justin Bogner via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 28 14:34:47 PDT 2024
bogner wrote:
Seeing some new warnings from clang after this change:
```
In file included from llvm/lib/Transforms/IPO/LowerTypeTests.cpp:51:
llvm/include/llvm/IR/ModuleSummaryIndexYAML.h:288:49: error: missing field 'Refs' initializer [-Werror,-Wmissing-field-initializers]
/*Aliasee=*/ASum->getAliaseeGUID()});
^
1 error generated.
```
These are visible on the pre-checkin job as well if you download the log: https://buildkite.com/llvm-project/github-pull-requests/builds/106415#01924ffe-ee6f-469e-98c3-e00581b5cd73
This looks benign but it might be worth explicitly defaulting the function summary members' initializations or giving `GlobalValueSummaryYaml` a constructor to make it clear which members are meant to be initialized on construction.
https://github.com/llvm/llvm-project/pull/106185
More information about the llvm-commits
mailing list