[PATCH] D74257: IR Linking: Support merging Warning+Max module metadata flags

[David Blaikie via Phabricator via llvm-commits]

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