This is an archive of the discontinued LLVM Phabricator instance.

[Compiler-rt][MIPS][Profile] Adding support for MIPS32/64
ClosedPublic

Authored by mohit.bhakkad on Aug 13 2014, 2:24 AM.

Details

Summary

Changed cmake/config-ix.cmake to add support for different MIPS architectures: mips, mipsel, mips64, mips64el
In profile code there is no target based dependencies, so just enabling mips flag does the work.
Test results are added below.

Diff Detail

Event Timeline

mohit.bhakkad retitled this revision from to [Profile] adding support for MIPS32/MIPS64.
mohit.bhakkad updated this object.
mohit.bhakkad edited the test plan for this revision. (Show Details)
mohit.bhakkad added reviewers: dsanders, void, petarj.
mohit.bhakkad set the repository for this revision to rL LLVM.
mohit.bhakkad added subscribers: Unknown Object (MLST), sdkie, farazs.
mohit.bhakkad edited the test plan for this revision. (Show Details)Sep 5 2014, 8:42 AM
mohit.bhakkad updated this object.
mohit.bhakkad set the repository for this revision to rL LLVM.

Added support for different MIPS architectures: mips, mipsel, mips64

mohit.bhakkad added inline comments.Sep 17 2014, 3:49 AM
cmake/config-ix.cmake
82

Defination of CMake if(MATCHES) command is
if(<variable|string> MATCHES regex)

Which can be ambiguous because of regex as an operand.

Whereas for CMake if(STREQUAL) exact lexicographical comparison between left and right operand is done;
if(<variable|string> STREQUAL <variable|string>)

source: http://www.cmake.org/cmake/help/v3.0/command/if.html

mohit.bhakkad retitled this revision from [Profile] adding support for MIPS32/MIPS64 to [Compiler-rt][MIPS][Profile] Adding support for MIPS32/64.
mohit.bhakkad updated this object.
mohit.bhakkad removed reviewers: kcc, petarj, void, dsanders.
mohit.bhakkad set the repository for this revision to rL LLVM.
mohit.bhakkad removed a subscriber: Unknown Object (MLST).
mohit.bhakkad set the repository for this revision to rL LLVM.
mohit.bhakkad added a subscriber: Unknown Object (MLST).
samsonov added inline comments.Sep 25 2014, 10:40 AM
cmake/config-ix.cmake
101–108

Why not just write "mipsel|mips64el" ?

mohit.bhakkad added inline comments.Sep 25 2014, 10:57 PM
cmake/config-ix.cmake
101–108

Yes, I will change it, and thinking about using STREQUAL instead of MATCHES which will result in faster comparison.

samsonov accepted this revision.Sep 26 2014, 1:49 PM
samsonov edited edge metadata.

LGTM

This revision is now accepted and ready to land.Sep 26 2014, 1:49 PM

I don't have commit access, can someone please commit it for me?

dsanders closed this revision.Oct 2 2014, 5:57 AM
dsanders edited edge metadata.Oct 2 2014, 5:58 AM

Committed in r218866.