[webkit-reviews] review requested: [Bug 130523] Need target to produce llvm ir : [Attachment 229002] Added target to produce llvm ir for JavaScriptCore. WIP for C++ inlining.

[bugzilla-daemon at webkit.org]

Matthew Mirman <mmirman at apple.com> has asked  for review:
Bug 130523: Need target to produce llvm ir
https://bugs.webkit.org/show_bug.cgi?id=130523

Attachment 229002: Added target to produce llvm ir for JavaScriptCore.	WIP for
C++ inlining.
https://bugs.webkit.org/attachment.cgi?id=229002&action=review

------- Additional Comments from Matthew Mirman <mmirman at apple.com>
> Why do we need to look at the version of the file with a .o suffix? Is that
because the .bc file has the wrong modification time due to being copied? Can
we switch to using "cp -p" to preserve the modification time on the .bc file,
then ignore the .o files in this script?

I'm using the .o version (which is a binary object file, and not an llvm
bitcode file), so that I can get its symbol table using "nm" .	The
modification time is just a perk.