[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