[PATCH] D17472: [NVPTX] Annotate some instructions as hasSideEffects = 0.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 18:56:20 PST 2016


jlebar created this revision.
jlebar added a reviewer: jholewinski.
jlebar added subscribers: llvm-commits, tra.
Herald added a subscriber: jholewinski.

Tablegen tries to infer this from the selection DAG patterns defined for
the instructions, but it can't always.

An instructive example is CLZr64.  CLZr32 is correctly inferred to have
no side-effects, but the selection DAG pattern for CLZr64 is slightly
more complicated, and in particular the ctlz DAG node is not at the root
of the pattern.  Thus tablegen can't infer that CLZr64 has no
side-effects.

http://reviews.llvm.org/D17472

Files:
  lib/Target/NVPTX/NVPTXInstrInfo.td
  lib/Target/NVPTX/NVPTXIntrinsics.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17472.48568.patch
Type: text/x-patch
Size: 18202 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160220/e18e8744/attachment-0001.bin>


More information about the llvm-commits mailing list