[PATCH] D77824: [mlir] Emit errors if global constructors are found within lib/

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: mlir/lib/IR/CMakeLists.txt:2
+# TODO: Remove the need for global constructors within IR/
+add_flag_if_supported("-Wno-global-constructors" WNO_GLOBAL_CONSTRUCTOR_MLIR_IR)
+
----------------
bondhugula wrote:
> This should be added to test/lib/ as well?
We could, but we don't really have a reason to limit what happens in tests do we?

The motivation for doing this right now is about preserving the ability to build and embed the compiler in a product without having these ctors.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77824/new/

https://reviews.llvm.org/D77824