[PATCH] D74257: IR Linking: Support merging Warning+Max module metadata flags
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 7 16:32:10 PST 2020
dblaikie marked an inline comment as done.
dblaikie added inline comments.
================
Comment at: llvm/lib/Linker/IRMover.cpp:1357
+
+ if (DstBehaviorValue == Module::Max || SrcBehaviorValue == Module::Max) {
+ ConstantInt *DstValue =
----------------
tejohnson wrote:
> A comment about why this isn't in the switch statement would be helpful, along with a note about how the new Flag is formed.
>
> Also, it might be clearer to put this before the switch, after the new warning handling, with a continue to skip the then unnecessary switch (it consolidates the special case handling for Warning+Max).
Makes sense, thanks! Did those.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74257/new/
https://reviews.llvm.org/D74257
More information about the llvm-commits
mailing list