[PATCH] D84689: [OpenMPOpt] Most SCC's are uninteresting, don't waste time on them (up to 16x faster)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri updated this revision to Diff 281040.
lebedev.ri retitled this revision from "[OpenMPOpt][OldPM!] Most SCC's are uninteresting, don't waste time on them" to "[OpenMPOpt] Most SCC's are uninteresting, don't waste time on them (up to 16x faster)".
lebedev.ri edited the summary of this revision.
lebedev.ri added a comment.

As per IRC disscussion, let's do the (identical!) check in another way,
by recording in a set which functions are known to have openmp runtime calls,
and simply looking each function in SCC in that set.
This appears to be identical in the sense of detection
(though, if the omp functions are used only by constantexprs, we'll miss those cases.),
and at least not worse performance-wise.

As a bonus, works for NewPM!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84689/new/

https://reviews.llvm.org/D84689

Files:
  llvm/include/llvm/Transforms/IPO/OpenMPOpt.h
  llvm/lib/Transforms/IPO/OpenMPOpt.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84689.281040.patch
Type: text/x-patch
Size: 4043 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200727/9ea32a97/attachment.bin>