[PATCH] Lower invoke statepoints and gc.results tied to them.

Igor Laevsky igor at azulsystems.com
Tue Mar 3 13:06:16 PST 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/CodeGen/SelectionDAG/StatepointLowering.cpp:258
@@ +257,3 @@
+      // We can not use standart mechanism for exporting values here
+      // (which is 'ExportFromCurrentBlock'). Problem is that exported register
+      // should be tied to the llvm-ir Value*, and value we want to export
----------------
reames wrote:
> Given our discussion, this comment is wrong and misleading.  It's not the fact that "Tmp" is deleted, it's the fact that the type will not match.  Please update.
It depends on the view point. For 'Tmp' types actually match. So we can look at the problem either as standart mechanism not knowing about statepoints or as our inability to export correct Value*. Anyway it is probably unnecessary details, I updated the comment.

http://reviews.llvm.org/D7760

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list