[PATCH] D54343: [llvm-exegesis][NFC] Some code style cleanup

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 12:14:27 PST 2018


jsji created this revision.
jsji added reviewers: courbet, gchatelet.
Herald added subscribers: tschuett, javed.absar.

Apply review comments of https://reviews.llvm.org/D54185 to other target as well, specifically:

1. make anonymous namespaces as small as possible, avoid using static inside anonymous namespaces
2. Add missing header to some files
3. GetLoadImmediateOpcodem-> getLoadImmediateOpcode
4. Fix typo


Repository:
  rL LLVM

https://reviews.llvm.org/D54343

Files:
  llvm/tools/llvm-exegesis/lib/X86/Target.cpp
  llvm/unittests/tools/llvm-exegesis/AArch64/TargetTest.cpp
  llvm/unittests/tools/llvm-exegesis/X86/AnalysisTest.cpp
  llvm/unittests/tools/llvm-exegesis/X86/RegisterAliasingTest.cpp
  llvm/unittests/tools/llvm-exegesis/X86/TargetTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54343.173408.patch
Type: text/x-patch
Size: 24558 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181109/f6e44ca4/attachment.bin>


More information about the llvm-commits mailing list