[PATCH] D39390: Move isDSOLocal check and add a comment

[Sean Fertile via Phabricator via llvm-commits]

sfertile added a comment.

This LGTM.



================
Comment at: lib/Target/TargetMachine.cpp:117
                                          const GlobalValue *GV) const {
+  // If the front end requested that this GV be treated as dso local, obey.
+  if (GV && GV->isDSOLocal())
----------------
minor nit: it might not be the front-end that set the value, the lto backends could refine a GV to dso_local using the symbol resolutions the linker provides.


https://reviews.llvm.org/D39390