[llvm] r227584 - ARM: correct handling of .fpu directive

Saleem Abdulrasool compnerd at compnerd.org
Fri Jan 30 10:28:27 PST 2015


On Fri, Jan 30, 2015 at 10:20 AM, Hans Wennborg <hans at chromium.org> wrote:

> On Fri, Jan 30, 2015 at 9:58 AM, Saleem Abdulrasool
> <compnerd at compnerd.org> wrote:
> > Author: compnerd
> > Date: Fri Jan 30 11:58:25 2015
> > New Revision: 227584
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=227584&view=rev
> > Log:
> > ARM: correct handling of .fpu directive
> >
> > The FPU directive permits the user to switch the target FPU, enabling
> > instructions that would be otherwise unavailable.  However, when
> configuring the
> > new subtarget features, we would not enable the implied functions for
> newer
> > FPUs.  This would result in invalid rejection of valid input.  Ensure
> that we
> > inherit the implied FPU functionality when enabling newer versions of
> the FPU.
> > Fortunately, these are mostly hierarchical, unlike the CPUs.
> >
> > Addresses PR22395.
>
> Should we merge this to 3.6?
>

Good call, I think that is a good idea.

-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150130/da44da2a/attachment.html>


More information about the llvm-commits mailing list