[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