This is an archive of the discontinued LLVM Phabricator instance.

[X86][BtVer2][NFC] Regexify zero-idioms
AbandonedPublic

Authored by lebedev.ri on Jul 3 2018, 6:35 AM.

Details

Reviewers
RKSimon

Diff Detail

Repository
rL LLVM

Event Timeline

lebedev.ri created this revision.Jul 3 2018, 6:35 AM

Please don't use regexes unless you really need to - its a little difficult to gauge but in general these take a LOT longer on tblgen - they only really work well if you don't start with a pattern so InstRegexOp::apply can reduce the search range considerably.

dkrupp added a subscriber: dkrupp.Jul 3 2018, 7:02 AM
dkrupp removed a subscriber: dkrupp.
lebedev.ri abandoned this revision.Jul 3 2018, 11:02 AM

Please don't use regexes unless you really need to - its a little difficult to gauge but in general these take a LOT longer on tblgen - they only really work well if you don't start with a pattern so InstRegexOp::apply can reduce the search range considerably.

Interesting, unfortunate. Ok, good to know!