[PATCH] D31349: IR: Replace the "Linker Options" module flag with "llvm.linker.options" named metadata.
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 24 15:29:51 PDT 2017
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
After briefly attempting to rework https://reviews.llvm.org/D31352 to work without this patch, I now see why you want to do this. Working with module flags is hard and working with named md nodes is easier. Module flags still seem appropriate for integer settings that need to match during LTO, but I don't think there are many appending module flags that wouldn't be better served by named md nodes.
More information about the llvm-commits