Make llvm::call_once more convenient to reuse out of LLVM
ClosedPublic

Authored by krytarowski on Mon, Jan 30, 12:25 PM.

Details

Summary

Currently users need to set call using namespace llvm;, with this change it's no longer needed.

Sponsored by <The NetBSD Foundation>

Diff Detail

Repository
rL LLVM
krytarowski created this revision.Mon, Jan 30, 12:25 PM

Related review: D29288

If we want to make this namespace aware, I wonder if we shouldn't also be explicitly and use ::llvm::once_flag etc. But in general, LGTM.

LGTM, but I will let other LLVM folks give it the final approval.

The idea of this seems fine as-is, but it should be clang-formatted to fit in 80 columns.

krytarowski updated this revision to Diff 86740.Wed, Feb 1, 3:32 PM

Break line to fit the 80-cols limit.

LGTM.

I'd go with @joerg suggestion though (but you don't need to submit another patch, just go ahead and commit).

mehdi_amini accepted this revision.Thu, Feb 2, 8:14 AM
This revision is now accepted and ready to land.Thu, Feb 2, 8:14 AM
krytarowski updated this revision to Diff 86818.Thu, Feb 2, 8:16 AM

Use ::lvm namespace.

krytarowski closed this revision.Thu, Feb 2, 8:17 AM