[PATCH] D53758: Handle value uses wrapped in metadata for the use-list order
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 8 10:41:57 PST 2018
dexonsmith added inline comments.
================
Comment at: test/Assembler/metadata-use-uselistorder.ll:5-6
+
+; Verify that the uses of i64 0 in the llvm.dbg.value operands are considered
+; when generating the use-list order.
+
----------------
Does this only cause problems for constants (like `i64 0`), or also for use lists of locals and globals?
The reason I ask: we *should* make it impossible for most constants to rely on use-list order, and then stop storing use-lists for them (in either bitcode or textual IR). When we get there, should we delete this test and the new code you've added? Or will it still be relevant for other values?
https://reviews.llvm.org/D53758
More information about the llvm-commits
mailing list