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

Mark emitIns_R_A and emitIns_R_R_A to be not defined for legacy backend #15880

Merged
merged 1 commit into from
Jan 16, 2018
Merged

Mark emitIns_R_A and emitIns_R_R_A to be not defined for legacy backend #15880

merged 1 commit into from
Jan 16, 2018

Conversation

tannergooding
Copy link
Member

@tannergooding
Copy link
Member Author

Working on pulling down the TFS sources to validate locally.

CarolEidt
CarolEidt approved these changes Jan 16, 2018
View reviewed changes
Copy link

@CarolEidt CarolEidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tannergooding
Copy link
Member Author

@sandreenko, this looks to be building successfully on my end (provided you also disable FEATURE_HW_INTRINSICS for the legacy backend).

@sandreenko
Copy link

sandreenko commented Jan 16, 2018 •
edited
Loading

@sandreenko, this looks to be building successfully on my end (provided you also disable FEATURE_HW_INTRINSICS for the legacy backend).

Great, merge it.

@tannergooding tannergooding merged commit 4f2f9be into dotnet:master Jan 16, 2018
@tannergooding tannergooding deleted the hwintrin-containment branch January 17, 2018 01:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@CarolEidt CarolEidt CarolEidt approved these changes

@sandreenko sandreenko sandreenko approved these changes

Assignees
No one assigned
Labels
area-CodeGen
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tannergooding @sandreenko @CarolEidt @jkotas