This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add missing namespace (NFC)
ClosedPublic

Authored by fakepaper56 on Jan 13 2022, 1:31 AM.

Diff Detail

Event Timeline

fakepaper56 created this revision.Jan 13 2022, 1:31 AM
fakepaper56 requested review of this revision.Jan 13 2022, 1:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2022, 1:31 AM
clang/utils/TableGen/RISCVVEmitter.cpp
300

We have used
using namespace llvm;

The main reason I want to add the prefix "llvm::" is all None in the files are "llvm::None" form. But also it is more feasible when we want to add a variable name 'None' in the file.

The main reason I want to add the prefix "llvm::" is all None in the files are "llvm::None" form.

It makes sense to me.

This revision is now accepted and ready to land.Jan 13 2022, 5:17 PM
fakepaper56 closed this revision.Feb 23 2022, 9:45 PM

The patch content is in D119727.