[PATCH] D37617: Debug info for variables whose type is shrinked to bool fix

[Nikola Prica via Phabricator via llvm-commits]

NikolaPrica closed this revision.
NikolaPrica added inline comments.


================
Comment at: lib/Transforms/IPO/GlobalOpt.cpp:1648
+     // FIXME: This will only emit address for debugger on which will
+     // be written only 0 or 1.
+     for(auto *GV : GVs)
----------------
aprantl wrote:
> Why this restriction?
This condition is necessary for examples like:
 
   long a[];
   static int b = a;

Previous version of this patch did not check for dyn_cast for CIInit which resulted in failure for the example above.


https://reviews.llvm.org/D37617