[PATCH] D26045: [cfi] Fix weak functions handling.

[Evgeniy Stepanov via llvm-commits]

eugenis added inline comments.


================
Comment at: lib/Transforms/IPO/LowerTypeTests.cpp:747
+            : ".text.startup");
+    appendToGlobalCtors(M, WeakInitializerFn, /* Priority */ 0);
+  }
----------------
I wonder if the priority should be lower. Regular constructors have priority of 65535, we definitely want higher than that, but maybe lower than ASan, etc constructors which use 1 and 2 currently.


Repository:
  rL LLVM

https://reviews.llvm.org/D26045