[PATCH] D15556: [RS4GC] Fix crash in the case that a live variable has a constant base.
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 22 10:33:54 PST 2015
reames added inline comments.
Comment at: test/Transforms/RewriteStatepointsForGC/base-pointers-12.ll:17
@@ +16,3 @@
+declare void @extern() gc "statepoint-example"
You can put them in the same file or a different one as convient. No strong preference.
Sounds like we might want to consider adding a diagnostic flag to disable rematerialization. Doing so would make it easier to write the simple test and is probably worthwhile. Adding both the simple and complicated tests might be worth it as well.
By "dead", I meant "dynamically dead, but not statically dead". You can end up with all kinds of garbage down such a path because the compiler may have proven conflicting facts, but not yet realized the block is dead. Don't worry about this case specifically, I was mostly just throwing out ideas. I do want at least a bit more test coverage here, but I'm not picky about the form it takes.
p.s. The constant base assert has been removed.
More information about the llvm-commits