This is an archive of the discontinued LLVM Phabricator instance.

[libc][NFC] Remove headergen for the cacheline size macro.
ClosedPublic

Authored by sivachandra on Feb 24 2021, 12:13 AM.

Details

Summary

We want to be able to build and test the string functions in contexts
like that of Fuchsia where LLVM pieces like tablegen are not available.
Since header generation uses tablegen, we are removing the dependency on
headergen here.

Diff Detail

Event Timeline

sivachandra created this revision.Feb 24 2021, 12:13 AM
sivachandra requested review of this revision.Feb 24 2021, 12:13 AM
gchatelet accepted this revision.Feb 24 2021, 12:34 AM
gchatelet added inline comments.
libc/src/string/memory_utils/utils.h
9–10

Thx for the fix here.

26

Shall we add a supplementary else clause with an #error "unsupported platform" directive?

This revision is now accepted and ready to land.Feb 24 2021, 12:34 AM

Add default failure case.

sivachandra marked an inline comment as done.Feb 24 2021, 7:28 AM
This revision was landed with ongoing or failed builds.Feb 24 2021, 7:29 AM
This revision was automatically updated to reflect the committed changes.
libc/src/string/memory_utils/cacheline_size_arm.h.inc