[PATCH] D80714: [StatepointLowering] Handle UNDEF gc values.

Denis Antrushin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 05:57:26 PDT 2020


dantrushin created this revision.
dantrushin added reviewers: reames, skatkov, anna.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.
dantrushin marked an inline comment as done.
dantrushin added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GCStrategy.h:77
 
-  bool NeededSafePoints = false;    ///< if set, calls are inferred to be safepoints
+  bool NeededSafePoints = false; ///< if set, calls are inferred to be safepoints
   bool UsesMetadata = false;     ///< If set, backend must emit metadata tables.
----------------
Should this be spelled `NeedsSafePoints` (needs vs. needed) ?


Add a hook to GCStrategy telling if GC wants to relocate UNDEF
values. If it does not, we can skip spilling/relocating them.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D80714

Files:
  llvm/include/llvm/CodeGen/GCStrategy.h
  llvm/lib/CodeGen/SelectionDAG/StatepointLowering.cpp
  llvm/test/CodeGen/X86/statepoint-uniqueing.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80714.266827.patch
Type: text/x-patch
Size: 6044 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200528/338637b8/attachment.bin>


More information about the llvm-commits mailing list