[PATCH] D148953: [llvm][ADT] Fix Any linker error with multiple compilers

Joe Loser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 12:54:22 PDT 2023


jloser accepted this revision.
jloser added a comment.
This revision is now accepted and ready to land.

Thanks for the quick fix from the discussion on the issue today.

I'll be glad once we're off `llvm::Any` once and for all. :)



================
Comment at: llvm/include/llvm/ADT/Any.h:128
+// Define the type id and initialize with a non-zero value.
+// Initializing with a zero value means the variab can end up in either the
+// .data or the .bss section. This can lead to multiple definition linker errors
----------------
**Quibble**: `s/variab/variable`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148953



More information about the llvm-commits mailing list