[PATCH] ARM: Fix codegen for rbit intrinsic
Tim Northover
t.p.northover at gmail.com
Wed Aug 20 02:07:52 PDT 2014
Hi Yi,
Looks like an obvious change, but could you add/modify a test where the input argument actually *is* a constant integer? (It should still be fine because it's a Constant rather than a TargetConstant in the DAG, but it's as well to have it tested).
You can just commit with that adjustment.
Cheers.
Tim.
http://reviews.llvm.org/D4980
More information about the llvm-commits
mailing list