Page MenuHomePhabricator

[GVN] Remove duplicated canCoerce tests
AbandonedPublic

Authored by vtjnash on Nov 6 2019, 10:17 AM.

Details

Reviewers
reames
Summary

Now that canCoerceMustAliasedValueToLoad should contain the superset of all requirements for coercion to be legal to consider, we can now remove what should be duplicate tests scattered elsewhere.

Diff Detail

Event Timeline

vtjnash created this revision.Nov 6 2019, 10:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 6 2019, 10:17 AM
vtjnash updated this revision to Diff 277662.Jul 13 2020, 10:09 PM
vtjnash retitled this revision from [GVN] Remove duplicated canCoerce tests [NFCI] to [GVN] Remove duplicated canCoerce tests, NFCI.
vtjnash edited the summary of this revision. (Show Details)
vtjnash retitled this revision from [GVN] Remove duplicated canCoerce tests, NFCI to [GVN] Remove duplicated canCoerce tests [NFCI].Oct 15 2020, 11:42 AM
vtjnash retitled this revision from [GVN] Remove duplicated canCoerce tests [NFCI] to [GVN] Remove duplicated canCoerce tests.

Removed NFCI, since as I look back, I think this also enabled optimization of constant structures, and I was misusing the NFCI term

vtjnash abandoned this revision.Tue, Nov 3, 10:06 AM