[PATCH] [x86] Make vector legalization of extloads work more like the "normal" vector operation legalization with support for custom target lowering and fallback to expand when it fails, and use this to implement sext and anyext load lowering for x86 in a...

Chandler Carruth chandlerc at gmail.com
Thu Jul 24 06:14:43 PDT 2014


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:12914
@@ +12913,3 @@
+        SDValue(Ld, 1),
+        DAG.getNode(ISD::TokenFactor, dl, MVT::Other, Load.getValue(1)));
+
----------------
hfinkel at anl.gov wrote:
> Why are you creating a TokenFactor with one operand? Can't you just use the chain directly?
\meme{i have no idea what i'm doing}

Are you saying I should RAUW the old chain value with the chain value of the load? That makes sense if so, and I'll do so.

http://reviews.llvm.org/D4654






More information about the llvm-commits mailing list