[llvm-commits] [llvm] r73887 - /llvm/trunk/lib/Target/ARM/ARMISelDAGToDAG.cpp

[Anton Korobeynikov]

Hello, Bob

> Yes, I wondered about that, too.  It surprised me that a constant pool
> entry would be needed in ARM mode, but not in Thumb mode.  I'm in the
> middle of merging some changes and discovered that the llvm-gcc build
> failed compiling libgcc for ARM v7.  (The error message was "cannot
> select -1023" or something like that.) I had not seen this before so I
> replaced this check with the corresponding code that I had been using
> before.  It is definitely something that should be investigated.  Feel
> free to take a look; otherwise, I'll check it out later.
Ah, right. Currently movt is provided only for Thumb2 targets. We need
to add this entry to ARM side too. If seems we cannot "merge" these
two definitions: different encodings, in ARM mode movt can be
predicated.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University