Skip to content
This repository was archived by the owner on Apr 23, 2020. It is now read-only.

Fixed inclusion of M_PI fow MinGW-w64 #64

Closed
wants to merge 1 commit into from
Closed

Fixed inclusion of M_PI fow MinGW-w64 #64

wants to merge 1 commit into from

Conversation

KOLANICH
Copy link

@KOLANICH KOLANICH commented Nov 2, 2018

No description provided.

@sylvestre
Copy link
Collaborator

Sorry but we don't use github for contribution. Please see https://reviews.llvm.org/

@sylvestre sylvestre closed this Nov 2, 2018
@KOLANICH
Copy link
Author

KOLANICH commented Nov 2, 2018

@sylvestre could you pass this PR into the relevant place, please? I'm not going to register anywhere only to send several PRs fixing build issues.

@sylvestre
Copy link
Collaborator

Yeah, it is trivial enough. I landed the patch in r346000.

@KOLANICH
Copy link
Author

KOLANICH commented Nov 2, 2018

Thank you.

dtzWill pushed a commit that referenced this pull request Jul 19, 2019
Summary:
According to the new Armv8-M specification
https://static.docs.arm.com/ddi0553/bh/DDI0553B_h_armv8m_arm.pdf the
instructions SQRSHRL and UQRSHLL now have an additional immediate
operand <saturate>. The new assembly syntax is:

SQRSHRL<c> RdaLo, RdaHi, #<saturate>, Rm
UQRSHLL<c> RdaLo, RdaHi, #<saturate>, Rm

where <saturate> can be either 64 (the existing behavior) or 48, in
that case the result is saturated to 48 bits.

The new operand is encoded as follows:
  #64 Encoded as sat = 0
  #48 Encoded as sat = 1
sat is bit 7 of the instruction bit pattern.

This patch adds a new assembler operand class MveSaturateOperand which
implements parsing and encoding. Decoding is implemented in
DecodeMVEOverlappingLongShift.

Reviewers: ostannard, simon_tatham, t.p.northover, samparker, dmgreen, SjoerdMeijer

Reviewed By: simon_tatham

Subscribers: javed.absar, kristof.beyls, hiraditya, pbarrio, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D64810

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@366555 91177308-0d34-0410-b5e6-96231b3b80d8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants